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
helm: remove chart content not in common.yaml #8884
Conversation
metadata: | ||
name: rook-ceph-system-psp-users | ||
labels: | ||
operator: rook | ||
storage-backend: ceph | ||
chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}" | ||
roleRef: | ||
apiGroup: rbac.authorization.k8s.io | ||
kind: ClusterRole | ||
name: rook-ceph-system-psp-user | ||
subjects: | ||
- kind: ServiceAccount | ||
name: rook-ceph-system | ||
namespace: {{ .Release.Namespace }} | ||
--- | ||
apiVersion: rbac.authorization.k8s.io/v1 | ||
kind: ClusterRoleBinding |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rook-ceph-psp-user
doesn't exist any more. This seems to be an old vestigial remain.
- apiGroups: | ||
- monitoring.coreos.com | ||
resources: | ||
- servicemonitors | ||
- prometheusrules | ||
verbs: | ||
- get | ||
- list | ||
- watch | ||
- create | ||
- update | ||
- delete |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is also vestigial and covered by monitoring-specific resources that now exist.
e770238
to
d4ce7ef
Compare
@Mergifyio rebase |
Assume common.yaml is the right source of truth, and remove major content from helm charts that does not exist in common.yaml. Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
Command
|
d4ce7ef
to
507dfe1
Compare
@travisn this is ready for final review |
@Mergifyio backport |
Command
|
helm: remove chart content not in common.yaml (backport #8884)
Description of your changes:
Assume common.yaml is the right source of truth, and remove major
content from helm charts that does not exist in common.yaml.
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen
) has been run to update object specifications, if necessary.