Skip to content

atenet: drop redundant deployment mode - #635

Open
John Howard (howardjohn) wants to merge 1 commit into
agent-substrate:mainfrom
howardjohn:net/drop-managed-envoy
Open

atenet: drop redundant deployment mode#635
John Howard (howardjohn) wants to merge 1 commit into
agent-substrate:mainfrom
howardjohn:net/drop-managed-envoy

Conversation

@howardjohn

Copy link
Copy Markdown
Collaborator

This is intended to be essentially a dead code cleanup, not a controversial change. There were two ways to deploy the same proxy component; I assume no one is customizing the deployment mechanic, and this follows how other components work (just deployed with YAML).

This simplifies the setup so we only need to maintain one path. The default option is retained so there is no behavioral changes.

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

This is intended to be essentially a dead code cleanup, not a
controversial change. There were two ways to deploy the same proxy
component; I assume no one is customizing the deployment mechanic, and
this follows how other components work (just deployed with YAML).

This simplifies the setup so we only need to maintain one path. The
default option is retained so there is no behavioral changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant