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

Update authorization 1.10 files to work with status logic #489

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

jooseppi-luna
Copy link
Contributor

@jooseppi-luna jooseppi-luna commented Feb 22, 2024

Description

In the v1.4.2 patch release, labels were added to the config files for authorization so that we could process them in the csm status calculation logic. However, these labels were not in the 1.10 config files -- this PR adds them.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#1091

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have maintained backward compatibility

How Has This Been Tested?

Successfully installed auth proxy:

[root@master-1-Zaglt7mQUY8Wg e2e]# kubectl get pods -A
NAMESPACE           NAME                                                      READY   STATUS    RESTARTS      AGE
authorization       authorization-ingress-nginx-controller-7686cb6886-bvzkp   1/1     Running   0             48s
authorization       cert-manager-765754f9cd-z5knk                             1/1     Running   0             48s
authorization       cert-manager-cainjector-759bbd747b-l7zt9                  1/1     Running   0             48s
authorization       cert-manager-webhook-6fd48c65c8-99qkh                     1/1     Running   0             48s
authorization       proxy-server-56fc569dc8-wh6mw                             3/3     Running   0             51s
authorization       redis-commander-d55d67d46-6mptc                           1/1     Running   0             50s
authorization       redis-primary-65ddb4dcdd-p8r5r                            1/1     Running   0             51s
authorization       role-service-54fccd94d6-xtpqk                             1/1     Running   0             51s
authorization       storage-service-748bf79f4c-8cmtv                          1/1     Running   0             51s
authorization       tenant-service-6586cb867c-9sgkj                           1/1     Running   0             51s
dell-csm-operator   dell-csm-operator-controller-manager-779f6cdd47-j28fg     1/1     Running   0             7m59s
kube-flannel        kube-flannel-ds-5vmqz                                     1/1     Running   0             124d
kube-flannel        kube-flannel-ds-nwfs9                                     1/1     Running   0             124d
kube-flannel        kube-flannel-ds-z2875                                     1/1     Running   1 (92d ago)   124d
kube-system         coredns-5d78c9869d-5p48t                                  1/1     Running   1 (92d ago)   124d
kube-system         coredns-5d78c9869d-kkntt                                  1/1     Running   1 (92d ago)   124d
kube-system         etcd-master-1-zaglt7mquy8wg.domain                        1/1     Running   2 (72d ago)   124d
kube-system         kube-apiserver-master-1-zaglt7mquy8wg.domain              1/1     Running   2 (72d ago)   124d
kube-system         kube-controller-manager-master-1-zaglt7mquy8wg.domain     1/1     Running   1 (92d ago)   124d
kube-system         kube-proxy-gf2ws                                          1/1     Running   0             124d
kube-system         kube-proxy-k2sch                                          1/1     Running   1 (92d ago)   124d
kube-system         kube-proxy-tq6nr                                          1/1     Running   0             124d
kube-system         kube-scheduler-master-1-zaglt7mquy8wg.domain              1/1     Running   1 (92d ago)   124d
kube-system         snapshot-controller-55687d7977-2lhqg                      1/1     Running   1 (92d ago)   96d
kube-system         snapshot-controller-55687d7977-x46c6                      1/1     Running   0             96d
minio               minio-0                                                   1/1     Running   0             111d
minio               minio-1                                                   1/1     Running   0             111d
minio               minio-2                                                   1/1     Running   0             111d
minio               minio-3                                                   1/1     Running   0             111d
[root@master-1-Zaglt7mQUY8Wg e2e]# kubectl get csm -A
NAMESPACE       NAME            CREATIONTIME   CSIDRIVERTYPE   CONFIGVERSION   STATE
authorization   authorization   56s                                            Succeeded
[root@master-1-Zaglt7mQUY8Wg e2e]#

@jooseppi-luna jooseppi-luna merged commit f2e47b8 into main Feb 22, 2024
8 checks passed
@shanmydell shanmydell deleted the auth-update-config branch June 11, 2024 10:32
ChristianAtDell added a commit that referenced this pull request Oct 15, 2024
* add label to deployment

* add csm label to nginx deployment
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.

3 participants