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

clean: Update Glance Keystone Jobs to match common architecture - #281

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

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

Conversation

@intlabs

Copy link
Copy Markdown
Contributor

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

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

Notes for reviewers to consider:
This PR should bring Glance 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

@intlabsintlabs changed the title WIP: Update Glance Keystone Jobs to match common architecturefeat: Update Glance Keystone Jobs to match common architectureMar 15, 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.

As far as I can see, this LGTM @intlabs 👍

@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.

LTGM. good bye ansible woot!

@v1k0d3n
v1k0d3n merged commit e7f5b64 into att-comdev:masterMar 16, 2017
@v1k0d3n

Copy link
Copy Markdown
Collaborator

referenced as prerequisite to #289

@v1k0d3nv1k0d3n changed the title feat: Update Glance Keystone Jobs to match common architectureclean: Update Glance 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