Skip to content

HDDS-13875. Fixed wrong context reference in datanode-statefulset.yaml#21

Merged
adoroszlai merged 2 commits intoapache:mainfrom
DamianoErnesti-Anritsu:HDDS-13875
Nov 4, 2025
Merged

HDDS-13875. Fixed wrong context reference in datanode-statefulset.yaml#21
adoroszlai merged 2 commits intoapache:mainfrom
DamianoErnesti-Anritsu:HDDS-13875

Conversation

@DamianoErnesti-Anritsu
Copy link
Contributor

@DamianoErnesti-Anritsu DamianoErnesti-Anritsu commented Nov 3, 2025

What changes were proposed in this pull request?

This PR address a simple mistake in the value of storageClassName reference in datanode-statefulset.yaml

What is the link to the Apache JIRA

HDDS-13875

@adoroszlai
Copy link
Contributor

Thanks @DamianoErnesti-Anritsu for the patch, makes sense to me.

@dnskr can you please take a look?

Copy link
Contributor

@dnskr dnskr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR!
I found that om-statefulset.yaml and s3g-statefulset.yaml have the same issue. Would you like to address them in this PR as well?

@adoroszlai
Copy link
Contributor

I found that om-statefulset.yaml and s3g-statefulset.yaml have the same issue.

Nice catch.

address them in this PR as well?

Yes, please. No need for new issue + PR for each file.

@DamianoErnesti-Anritsu
Copy link
Contributor Author

Thank you for the PR! I found that om-statefulset.yaml and s3g-statefulset.yaml have the same issue. Would you like to address them in this PR as well?

Yeah, kinda figured it was worth a check yesterday but was eventually caught with work and forgot, thank you for noticing!

I found that om-statefulset.yaml and s3g-statefulset.yaml have the same issue.

Nice catch.

address them in this PR as well?

Yes, please. No need for new issue + PR for each file.

Changes done, everything should be okay now.

@adoroszlai adoroszlai merged commit 015b128 into apache:main Nov 4, 2025
1 check passed
@adoroszlai
Copy link
Contributor

Thanks @DamianoErnesti-Anritsu for the fix, @dnskr for the review.

@morooshka
Copy link

Great to see this merged. Is there a projected release date for the next version? Thank you

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.

4 participants