-
Notifications
You must be signed in to change notification settings - Fork 560
Remove name
opt from create/update opts
#3017
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
base: main
Are you sure you want to change the base?
Remove name
opt from create/update opts
#3017
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for submitting your first PR! Be sure that we will be looking at it but keep in mind
this sometimes takes a while.
Please let the maintainers know if your PR has not got enough attention after a few days.
If any doubt, please consult our PR tutorial.
602f291
to
8a16433
Compare
here is my recent devstack installation, the names are there:
|
Different API. You want >>> import openstack
>>> conn = openstack.connect()
>>> conn.identity.get('/endpoints').json()['endpoints'][0].keys()
dict_keys(['id', 'interface', 'region_id', 'service_id', 'url', 'enabled', 'region', 'links']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming CI and @kayrus are happy.
hm, I executed |
Yeah, that's a meta command that takes the output of the services API and combined it with the output of the endpoint API. If you search higher in the debug output you'll likely see the call to the latter. |
Fixes #3016
Remove
name
opt from create/update opts because the property is not in endpoint schema (1).Also,
name
is not in keystone api v3 document (2).Links to the line numbers/files in the OpenStack source code that support the code in this PR: