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

[FEATURE]: Remove CSM for Authorization Sidecar injection in favor of Helm chart deployment #112

Closed
atye opened this issue Nov 29, 2021 · 0 comments
Assignees
Labels
area/csm-authorization Issue pertains to the CSM Authorization module type/feature A feature. This label is applied to a feature issues.
Milestone

Comments

@atye
Copy link
Contributor

atye commented Nov 29, 2021

Describe the solution you'd like
Remove sidecar injection in favor of the deployment via the driver helm chart.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@atye atye added type/feature-request New feature request. This is the default label associated with a feature request issue. area/csm-authorization Issue pertains to the CSM Authorization module labels Nov 29, 2021
@atye atye self-assigned this Nov 29, 2021
@gallacher gallacher changed the title [FEATURE]: Sidecar injection should be deprecated [FEATURE]: CSM for Authorization Sidecar injection should be deprecated Nov 30, 2021
@gallacher gallacher added this to the v1.2.0 milestone Dec 7, 2021
@gallacher gallacher added type/feature A feature. This label is applied to a feature issues. and removed type/feature-request New feature request. This is the default label associated with a feature request issue. labels Dec 8, 2021
@gallacher gallacher changed the title [FEATURE]: CSM for Authorization Sidecar injection should be deprecated [FEATURE]: Remove CSM for Authorization Sidecar injection in favor of Helm chart deployment Mar 16, 2022
csmbot pushed a commit that referenced this issue Aug 1, 2023
* [replication] Added upgrade page and updated install info (#57)

* Added note about repctl logs file

* Added upgrade instructions for both controller and sidecar

* modified installation\upgrade section

* Fixed couple of grammar mistakes

* Added new entry to troubleshooting page

* Addressed review comments

* Changed link address

Co-authored-by: Maxim Sklyarov <[email protected]>

* Update deployment steps for CSM Authorization (#58)

* begin updating deployment

* fixed typos

* add auth upgrade doc

* updated powerscale with authorization

* updated authorization documentation for powermax, powerflex, and powerscale

* refactored for powermax

* added vxflexos related docs for auth deployment and configuration

* consolidated proxy server root cert

* fix grammar, notes, value.yaml parameters, update auth deployment

* added note for driver configurations with auth

* updated note

* add auth note to drivers

* update upgrade path

Co-authored-by: atye <[email protected]>
Co-authored-by: sharmilarama <[email protected]>
Co-authored-by: Logan Jones <[email protected]>

* Fix operator install docs (#62)

* Small update to the contributing doc (#54)

* Update _index.md

* Update _index.md

* fixed sidecar instructions

* Update _index.md

* making changes requested by Aron

* trying to get rid of unwanted changes

Co-authored-by: gallacher <[email protected]>

* add Volume Health Monitor section (#67)

* add Volume Health Monitor section

* PR feedback

* pv/pvc metrics csi-powerstore changes (#64)

* Added troubleshooting documentation about gateway timeout for authorization (#63)

* Upgrade and Rollback Support for CSM for Authorization proxy server (#66)

* added auth upgrade and rollback, updated auth notes for drivers

* fixed spacing

* [replication] Added uninstall page, updated repctl readme (#70)

* static provisioning and ephemeral changes (#71)

* Update uninstall.md

* updated auuth deployment steps (#72)

* add  healthMonitorInterval to values table (#79)

* Helm install update (#74)

* updating helm install instructions

* adding troubleshooting for helm update

* minor changes and updates

* more minor changes

* word change

* more minor changes

* addressing comments from Jacob

* fixing numbers

* update code owners (#76)

* Move health monitor section to correct file  (#81)

* update correct file

* remove feature from wrong file

* Removed older OpenShift and added new driver versions (#84)

* Feature rwop csi powerstore (#89)

* Documentation for RWOP - CSI Powerstore

* Addressed review comment

* Update powerstore.md

Co-authored-by: shanmydell <[email protected]>

* Feature rwop accessmode support for csi-powerscale (#90)

Co-authored-by: shanmydell <[email protected]>

* Tenant documentation for both csi-unity and operator (#85)

Co-authored-by: shanmydell <[email protected]>

* Replication prerequisites & troubleshooting (#93)

Co-authored-by: shanmydell <[email protected]>

* Feature/pvc metrics csi powerstore update (#91)

* volume health monitoring update (#92)

* volume health monitoring update

* Update powerscale.md

* update documentation for health monitoring

Co-authored-by: shanmydell <[email protected]>
Co-authored-by: Randeep Sharma <[email protected]>
Co-authored-by: Bahubali Jain <[email protected]>

* Changed replication support matrix (#94)

* Changed replication support matrix

* Changed to X

* Add health values (#95)

* add new values to values table

* Add note to features section

* fix typo

* Common changes (#86)

* Unity - RWOP Access Mode and Volume Health Monitoring (#77)

* RWOP support matrix change (#96)

* Added known issue for unity (#97)

* Update powerflex.md (#98)

* powerscale release notes updated (#99)

* Operator Docs changes related to Unity features (#102)

* Operator upgrade documentation for volume health monitor changes (#104)

* Added note about how to list volume snapshots (#101)

* restructured deployment docs (#106)

* Improve operator install steps (#107)

* Update versions (#100)

* Added note that clarifies keys for csm installer (#108)

* Added volume health monitor in CSI spec support (#109)

* updated sample update for topology usage (#112)

#82

Co-authored-by: Andrey Schipilo <[email protected]>
Co-authored-by: Maxim Sklyarov <[email protected]>
Co-authored-by: shaynafinocchiaro <[email protected]>
Co-authored-by: atye <[email protected]>
Co-authored-by: sharmilarama <[email protected]>
Co-authored-by: Logan Jones <[email protected]>
Co-authored-by: Jooseppi Luna <[email protected]>
Co-authored-by: JacobGros <[email protected]>
Co-authored-by: Ashish Verma <[email protected]>
Co-authored-by: Trevor Dawe <[email protected]>
Co-authored-by: gilltaran <[email protected]>
Co-authored-by: hoppea2 <[email protected]>
Co-authored-by: Francis Nijay <[email protected]>
Co-authored-by: shanmydell <[email protected]>
Co-authored-by: Bahubali Jain <[email protected]>
Co-authored-by: karthikk92 <[email protected]>
Co-authored-by: Sakshi-dell <[email protected]>
Co-authored-by: Randeep Sharma <[email protected]>
Co-authored-by: Bahubali Jain <[email protected]>
Co-authored-by: rensyct <[email protected]>
Co-authored-by: Rajendra Indukuri <[email protected]>
Co-authored-by: abhi16394 <[email protected]>
Co-authored-by: panigs7 <[email protected]>
Co-authored-by: Prasanna M <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csm-authorization Issue pertains to the CSM Authorization module type/feature A feature. This label is applied to a feature issues.
Projects
None yet
Development

No branches or pull requests

3 participants