Skip to content

Update docs to point out admin port change #905

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

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

technowhizz
Copy link
Contributor

Points out the admin port change in keystone and mentions changing the cephadm.yml RGW config file as an example

@technowhizz technowhizz self-assigned this Jan 31, 2024
@technowhizz technowhizz requested a review from a team as a code owner January 31, 2024 15:46
@Alex-Welsh
Copy link
Member

Does it delete the endpoint in openstack or does it just stop working? i.e. if you did an openstack endpoint listwould it show up?

@technowhizz
Copy link
Contributor Author

Nope the endpoint is still there. It's the same port as the public (and internal) port

@Alex-Welsh
Copy link
Member

Nope the endpoint is still there. It's the same port as the public (and internal) port

Might be safest to add a step to remove it in that case. I'm not 100% sure but I think we can remove all the admin endpoints once we're on Zed.

@markgoddard
Copy link
Contributor

Nope the endpoint is still there. It's the same port as the public (and internal) port

Might be safest to add a step to remove it in that case. I'm not 100% sure but I think we can remove all the admin endpoints once we're on Zed.

I started this patch to remove the admin endpoint in bobcat, and was planning to backport to antelope. I'll see if I can push it along.

@markgoddard markgoddard merged commit e988956 into stackhpc/zed Feb 5, 2024
@markgoddard markgoddard deleted the zed-keystone-rgw branch February 5, 2024 12:16
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.

3 participants