Skip to content
This repository was archived by the owner on Jul 24, 2019. It is now read-only.

clean: Update Neutron Keystone Jobs to match common architecture - #275

Merged
v1k0d3n merged 2 commits into
att-comdev:masterfrom
portdirect:0.2.0/job-refactor-neutron
Mar 18, 2017
Merged

clean: Update Neutron Keystone Jobs to match common architecture#275
v1k0d3n merged 2 commits into
att-comdev:masterfrom
portdirect:0.2.0/job-refactor-neutron

Conversation

@intlabs

Copy link
Copy Markdown
Contributor

What is the purpose of this pull request?: To bring Neutron Keystone Jobs inline with other services.

What issue does this pull request address?: N/A

Notes for reviewers to consider:
This PR should bring Neutron Keystone Jobs inline with Cinder and Heat, and is part of the effort to standardize the common elements in charts prior to moving to OpenStack.

Specific reviewers for pull request:
@alanmeadows@v1k0d3n@larryrensing@wilkers-steve

@v1k0d3nv1k0d3n changed the title Update Neutron Keystone Jobs to match common architecturefeat: Update Neutron Keystone Jobs to match common architectureMar 14, 2017

@wilkers-stevewilkers-steve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM @intlabs. Thanks again for handling these 👍

@v1k0d3nv1k0d3n left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Good cleanup @intlabs

@v1k0d3n
v1k0d3n merged commit 627b8e3 into att-comdev:masterMar 18, 2017
@v1k0d3n

Copy link
Copy Markdown
Collaborator

referenced as prerequisite for #289

@v1k0d3nv1k0d3n changed the title feat: Update Neutron Keystone Jobs to match common architectureclean: Update Neutron Keystone Jobs to match common architectureMar 18, 2017
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@intlabs@v1k0d3n@wilkers-steve@mcnanci