-
Notifications
You must be signed in to change notification settings - Fork 510
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
Vpnaas: Show IKE policy details #786
Conversation
Build succeeded.
|
2837bf2
to
25fd854
Compare
Build succeeded.
|
@jtopjian This is ready for review |
Example to Show the details of a specific IKE policy by ID | ||
|
||
policy, err := ikepolicies.Get(client, "f2b08c1e-aa81-4668-8ae1-1401bcb0576c").Extract() | ||
if err != nil { |
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.
Small nit to fix the indenting here. I'm going to go ahead and merge this since everything else is good, but could you fix this in a future PR?
This inline documentation will be displayed as the official documentation at godoc.org, so we want to make sure it's correct.
@simonre Looks good - thanks! Just one small doc nit for a future PR. |
For #723
Links to the line numbers/files in the OpenStack source code that support the
code in this PR:
https://github.com/openstack/neutron-vpnaas/blob/058469e1b99b647537a5228c6a384d93df5484df/neutron_vpnaas/db/vpn/vpn_db.py#L367