Skip to content

CLOUDP-331537: Remove subresourceEnabled Helm value #261

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 15, 2025
Merged

CLOUDP-331537: Remove subresourceEnabled Helm value #261

merged 1 commit into from
Jul 15, 2025

Conversation

m1kola
Copy link
Contributor

@m1kola m1kola commented Jul 14, 2025 •
edited
Loading

Summary

It was introduced as a workaround for an OpenShift bug and is no longer needed.

Proof of Work

CI must be green.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes? (Docs PR)
  • Have you checked for release_note changes? (Release notes PR)

Reminder (Please remove this when merging)

  • Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
  • Our Short Guide for PRs: Link
  • Remember the following Communication Standards - use comment prefixes for clarity:
    • blocking: Must be addressed before approval.
    • follow-up: Can be addressed in a later PR or ticket.
    • q: Clarifying question.
    • nit: Non-blocking suggestions.
    • note: Side-note, non-actionable. Example: Praise
    • --> no prefix is considered a question

@m1kola m1kola requested a review from a team as a code owner July 14, 2025 15:41
@m1kola m1kola requested review from MaciejKaras and anandsyncs July 14, 2025 15:41
@m1kola m1kola force-pushed the clenup_subresource branch from 8d632d4 to f47b8fa Compare July 14, 2025 21:42
@m1kola
Remove subresourceEnabled Helm value
29903ac 
It was introduced as a workaround for an OpenShift bug
and is no longer needed.
@m1kola m1kola force-pushed the clenup_subresource branch from f47b8fa to 29903ac Compare July 15, 2025 10:57
@m1kola
Copy link
Contributor Author

m1kola commented Jul 15, 2025

Had to rebase on top of master to resolve conflict because helm_chart/templates/operator-roles.yaml is now helm_chart/templates/operator-roles-base.yaml after #258.

@m1kola m1kola changed the title Remove subresourceEnabled Helm value CLOUDP-331537: Remove subresourceEnabled Helm value Jul 15, 2025
@m1kola
Copy link
Contributor Author

m1kola commented Jul 15, 2025 •
edited
Loading

Release note for this in a separate PR: #266

m1kola added a commit to m1kola/mongodb-kubernetes that referenced this pull request Jul 15, 2025
@m1kola
Add release notes
e636f93 
Release notes for:
* mongodb#261
* mongodb#260
@m1kola m1kola mentioned this pull request Jul 15, 2025
Merged
3 tasks
@m1kola m1kola merged commit 1d13fb3 into mongodb:master Jul 15, 2025
35 checks passed
@m1kola m1kola deleted the clenup_subresource branch July 15, 2025 15:13
m1kola added a commit that referenced this pull request Jul 15, 2025
@m1kola
Add release notes
588fcb5 
Release notes for:
* #261
* #260
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaciejKaras MaciejKaras MaciejKaras approved these changes

@mircea-cosbuc mircea-cosbuc mircea-cosbuc approved these changes

@anandsyncs anandsyncs Awaiting requested review from anandsyncs anandsyncs is a code owner automatically assigned from mongodb/kubernetes-hosted

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@m1kola @MaciejKaras @mircea-cosbuc