Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

Conversation

@nshankar13
Copy link
Contributor

@nshankar13 nshankar13 commented Jun 7, 2021

Signed-off-by: nshankar13 [email protected]

Description: Remove envoy pattern in meshconfig to allow user to point to envoy image in any registry, not just the osm dockerhub image.

Affected area:

Functional Area
New Functionality [ ]
Documentation [ ]
Install [ ]
CLI Tool [ ]
Certificate Management [ ]
Control Plane [ ]
Ingress [ ]
Egress [ ]
Networking [ ]
Observability [ ]
SMI Policy [ ]
Sidecar Injection [X]
Security [ ]
Upgrade [ ]
Tests [ ]
CI System [ ]
Demo [ ]
Performance [ ]
Other [ ]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project?

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change?

@nshankar13 nshankar13 requested a review from a team as a code owner June 7, 2021 17:15
Copy link
Contributor

@allenlsy allenlsy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: can you update the PR description to clarify why we make this change?

@nshankar13 nshankar13 linked an issue Jun 7, 2021 that may be closed by this pull request
@nshankar13 nshankar13 force-pushed the remove_envoy_pattern branch from 1d6bbaa to 89cf8f0 Compare June 7, 2021 17:41
@nshankar13 nshankar13 force-pushed the remove_envoy_pattern branch from 89cf8f0 to 56d9248 Compare June 7, 2021 18:07
@nshankar13 nshankar13 merged commit 1febd22 into openservicemesh:main Jun 7, 2021
@nshankar13
Copy link
Contributor Author

@Mergifyio backport release-v0.9

@mergify
Copy link
Contributor

mergify bot commented Jun 7, 2021

Command backport release-v0.9: success

Backports have been created

@openservicemesh openservicemesh deleted a comment from mergify bot Jun 7, 2021
@openservicemesh openservicemesh deleted a comment from mergify bot Jun 7, 2021
@nshankar13
Copy link
Contributor Author

@Mergifyio backport release-v0.9

@mergify
Copy link
Contributor

mergify bot commented Jun 7, 2021

Command backport release-v0.9: success

Backports have been created

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove envoy sidecar image pattern

3 participants