Skip to content
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 IPSec Policy Details #773

Merged
merged 6 commits into from
Feb 27, 2018

Conversation

simonre
Copy link
Contributor

@simonre simonre commented Feb 19, 2018

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#L449

@coveralls
Copy link

coveralls commented Feb 19, 2018

Coverage Status

Coverage increased (+0.008%) to 73.554% when pulling e6fa841 on simonre:vpnaas-ipsecpolicy-get into 47e78c6 on gophercloud:master.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Feb 19, 2018

Build succeeded.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Feb 21, 2018

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

@simonre simonre changed the title [WIP][PENDING #768]Vpnaas: Show IPSec Policy Details [WIP]Vpnaas: Show IPSec Policy Details Feb 26, 2018
@theopenlab-ci
Copy link

theopenlab-ci bot commented Feb 26, 2018

Build succeeded.

@simonre simonre changed the title [WIP]Vpnaas: Show IPSec Policy Details Vpnaas: Show IPSec Policy Details Feb 26, 2018
@simonre
Copy link
Contributor Author

simonre commented Feb 26, 2018

@jtopjian This is ready for review

@jtopjian
Copy link
Contributor

LGTM!

@jtopjian jtopjian merged commit fbf9a3c into gophercloud:master Feb 27, 2018
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.

None yet

3 participants