Skip to content

Conversation

@thotz
Copy link
Collaborator

@thotz thotz commented May 7, 2024

Fixes: #984

Checklist

  • Added tests for features and functional changes
  • Public functions and types are documented
  • Standard formatting is applied to Go code
  • Is this a new API? Added a new file that begins with //go:build ceph_preview
  • Ran make api-update to record new APIs

New or infrequent contributors may want to review the go-ceph Developer's Guide including the section on how we track API Status and the API Stability Plan.

The go-ceph project uses mergify. View the mergify command guide for information on how to interact with mergify. Add a comment with @Mergifyio rebase to rebase your PR when github indicates that the PR is out of date with the base branch.

Fixes: ceph#984

Signed-off-by: Jiffin Tony Thottan <thottanjiffin@gmail.com>
@phlogistonjohn phlogistonjohn added the no-API This PR does not include any changes to the public API of a go-ceph package label May 7, 2024
@phlogistonjohn
Copy link
Collaborator

I had to take a look at the conversation in the issue but this change aligns with @anoopcs9 request. Change looks ok to me, but as this is not an API change I would like @anoopcs9 to do the official approving. :-)

Copy link
Collaborator

@ansiwen ansiwen left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of that! LGTM.

@mergify mergify bot merged commit be5dff0 into ceph:master May 7, 2024
@phlogistonjohn
Copy link
Collaborator

lol. well, it got two approvals anyway :-D

@ansiwen
Copy link
Collaborator

ansiwen commented May 7, 2024

I had to take a look at the conversation in the issue but this change aligns with @anoopcs9 request. Change looks ok to me, but as this is not an API change I would like @anoopcs9 to do the official approving. :-)

Oops, sorry, I reviewed at the same moment. I think this is fine, he also just said "remove it".

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.

TestRadosGWTestSuite/TestUserBucket is consistently failing in CI

3 participants