-
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: List IPSec Policies #772
Conversation
Build succeeded.
|
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.
@simonre I did a cursory scan of this PR and left a few comments. Let me know if you have any questions.
This looks good, but I'll do a more closer review when it has been rebased and such.
// the API. Filtering is achieved by passing in struct field values that map to | ||
// the IPSec policy attributes you want to see returned. | ||
type ListOpts struct { | ||
ID string `q:"id"` |
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.
Same comment here as with the Services ListOpts.
EncapsulationMode string `q:"encapsulation_mode"` | ||
EncryptionAlgorithm string `q:"encryption_algorithm"` | ||
PFS string `q:"pfs"` | ||
Lifetime Lifetime `q:"lifetime"` |
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.
Is this actually possible to query with?
"encapsulation_mode": "tunnel", | ||
"encryption_algorithm": "aes-128", | ||
"pfs": "group5", | ||
"project_id": "b4eedccc6fb74fa8a7ad6b08382b852b", |
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.
Same comment as with the Services List PR. If both ProjectID
and TenantID
are being returned, let's make sure both are in the results struct.
6fa93ce
to
d35b4b5
Compare
Build succeeded.
|
@jtopjian This is ready for review |
Build succeeded.
|
@simonre This looks good! Nice work on the list implementation. There are some nits in the |
This commit removes the flatmap package from tests since it has been deprecated.
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#L454