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

Release preparations for v0.23.0 #912

Merged
merged 4 commits into from
Aug 14, 2023
Merged

Release preparations for v0.23.0 #912

merged 4 commits into from
Aug 14, 2023

Conversation

ansiwen
Copy link
Collaborator

@ansiwen ansiwen commented Aug 11, 2023

Fixes: #892

@ansiwen
Copy link
Collaborator Author

ansiwen commented Aug 11, 2023

@phlogistonjohn I tried to use "promote" mode of apiage.py, but I couldn't get it to work, maybe I misunderstand the feature. So I did the move from preview to stable in api-status.json manually. I would like to add a remark in the release docs, so I will not try that again in couple of months. So what is the right way?

@phlogistonjohn
Copy link
Collaborator

phlogistonjohn commented Aug 11, 2023

I think you may have forgotten that the script cross checks the state of the Go files with the JSON. You edited the JSON only. The tool expects you to edit the Go files to fix the ceph_preview flags and then you can run make api-promote to automate the updating of the JSON file.

The tests are all failing because the JSON file now disagrees with the Go sources. :-)

IOW my workflow would roughly be:

  • make api-report-updates RESULTS_DIR=$PWD/_results
  • cat _results/updates-found.md
  • for each dir in report
    • $EDITOR $dir/<...files...> - remove ceph_preview build tags
    • make api-promote RESULTS_DIR=$PWD/_results - update json
    • git commit -sp - commit changes

I do like to see at least one commit per directory (e.g. rbd, cephfs, cephfs/admin, and so on...).

If you'd prefer I add this to the release docs, I'm happy to do it. But I'll let you run through the process to validate it in case I missed anything.

@phlogistonjohn
Copy link
Collaborator

README change looks good BTW.

@ansiwen
Copy link
Collaborator Author

ansiwen commented Aug 14, 2023

Doh. 🤦‍♂️ I completely forgot about the preview tag in the code.

Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -14,7 +11,7 @@ func TestMakeDirs(t *testing.T) {
defer fsDisconnect(t, mount)

dir1 := "/base/sub/way"
err := mount.MakeDirs(dir1, 0755)
err := mount.MakeDirs(dir1, 0o755)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm guessing your IDE did this? Not an issue, I'm just curious.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, my IDE runs gofmt when I save. So maybe that changed?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Seem like that's the case :-)

@ansiwen ansiwen added the no-API This PR does not include any changes to the public API of a go-ceph package label Aug 14, 2023
@mergify mergify bot merged commit 0209e5b into master Aug 14, 2023
13 of 15 checks passed
@mergify mergify bot deleted the pr/ansiwen/release-prep branch August 14, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-API This PR does not include any changes to the public API of a go-ceph package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

APIs pending stability updates in v0.23.0
3 participants