Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove all injected volumes when running remove-from-mesh/uninject #26782

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

tariq1890
Copy link
Contributor

@tariq1890 tariq1890 commented Aug 24, 2020

The uninject logic does not remove all of the volumes injected by proxy injector

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure

Pull Request Attributes

Please check any characteristics that apply to this pull request.

[ ] Does not have any changes that may affect Istio users.

@tariq1890 tariq1890 requested review from linsun and clyang82 August 24, 2020 22:12
@tariq1890 tariq1890 requested a review from a team as a code owner August 24, 2020 22:12
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Aug 24, 2020
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 24, 2020
@tariq1890 tariq1890 added release-notes-none Indicates a PR that does not require release notes. cherrypick/release-1.7 labels Aug 24, 2020
@istio-testing
Copy link
Collaborator

In response to a cherrypick label: new pull request created: #26784

williamaronli added a commit to williamaronli/istio that referenced this pull request Aug 25, 2020
* copymaster: (223 commits)
  Fix release notes to match schema (istio#26785)
  relocate role rbac manifests and use the correct label (istio#26787)
  Split out xds.FakeDiscoveryServer for use in configgen (istio#26774)
  Enhance central istiod: allow pass env from inject webhook, any kv, any order (istio#25968) (istio#25968)
  Add authorization to XDS connection (istio#26450)
  update bootstrap with wellknown names (istio#26797)
  Refactor SDS logic (istio#26655)
  remove all injected volumes when running remove-from-mesh/uninject (istio#26782)
  Always confirm install (even if customized) (istio#26776)
  Move XDS cache to model (istio#26779)
  Automator: update istio/api@master dependency in istio/istio@master (istio#26778)
  Optimize slice memory allocations at different places: (istio#26550)
  feat: istioctl operator dump - add revision flag (istio#26773)
  Cleanup XDS Plugins (istio#26724)
  Pilot: Config deep copy bug fix (istio#26752)
  Refactor proxy-config cmd into independent sub commands (istio#26641)
  Update Istio workload and Istio service dashboards (istio#25794)
  feat:istioctl x add-to-mesh and remove-from-mesh Should not affect OwnerReferences (istio#26721)
  fix serviceaccount mismatch issue for operator. (istio#26756)
  Increase timeout on traffic test (istio#26753)
  ...

Change-Id: I37d27efa9a2533947c67f6387ba4a0270807b3c6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. release-notes-none Indicates a PR that does not require release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants