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

Upgrade Stargate and improve doc comments for stargate.image #403

Merged
merged 3 commits into from
Feb 22, 2021

Conversation

jakerobb
Copy link
Contributor

@jakerobb jakerobb commented Feb 22, 2021

What this PR does:
Upgrades Stargate to 1.0.9
Adds a bit of clarity to the doc comment for stargate.version

This is in draft mode because the images for 1.0.9 have not yet been published to Docker Hub. The changes themselves should be final, but I need to test with a cluster using the actual images.

Which issue(s) this PR fixes:
None

Checklist

  • Changes manually tested
  • Chart versions updated (if necessary)
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@jakerobb jakerobb added enhancement New feature or request component:stargate labels Feb 22, 2021
@jakerobb jakerobb added this to the 1.0.0 milestone Feb 22, 2021
@jakerobb jakerobb self-assigned this Feb 22, 2021
@jakerobb jakerobb added this to In progress in K8ssandra via automation Feb 22, 2021
@jakerobb jakerobb marked this pull request as draft February 22, 2021 16:46
@jakerobb jakerobb changed the title improved doc comments for stargate.image Upgrade Stargate and improve doc comments for stargate.image Feb 22, 2021
@jakerobb
Copy link
Contributor Author

I sanity checked Stargate 1.0.9 against C* 3.11.10 and 4.0.0beta4. All is well; I think this is ready to merge.

@jakerobb jakerobb marked this pull request as ready for review February 22, 2021 21:13
Copy link
Contributor

@jdonenine jdonenine left a comment

Choose a reason for hiding this comment

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

I finished up my side of the testing on 4.0, looks good with the bump of version and the removal of the property from values.yaml.

I talked to John quickly about removing from values.yaml to give him the heads up there, so if you're done testing 3.x @jakerobb I think we're good.

K8ssandra automation moved this from In progress to Reviewer approved Feb 22, 2021
@jakerobb jakerobb merged commit a034d9d into main Feb 22, 2021
K8ssandra automation moved this from Reviewer approved to Done Feb 22, 2021
@jakerobb jakerobb deleted the stargate-version-values-doc-tweak branch February 22, 2021 21:20
jeffreyscarpenter pushed a commit to jeffreyscarpenter/k8ssandra that referenced this pull request Mar 10, 2021
…ra#403)

* improved doc comments for stargate.image

* Updated Stargate to 1.0.9

* Remove stargate.clusterVersion from values.yaml (k8ssandra#405)

Co-authored-by: Jeff DiNoto <jeff.dinoto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:stargate enhancement New feature or request
Projects
No open projects
K8ssandra
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants