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

Conversation

@intlabs
Copy link
Contributor

@intlabs intlabs commented Mar 15, 2017

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

@intlabs intlabs changed the title WIP: Update Glance Keystone Jobs to match common architecture feat: Update Glance Keystone Jobs to match common architecture Mar 15, 2017
Copy link
Contributor

@wilkers-steve wilkers-steve left a comment

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 👍

Copy link
Collaborator

@v1k0d3n v1k0d3n left a comment

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:master Mar 16, 2017
@v1k0d3n
Copy link
Collaborator

v1k0d3n commented Mar 18, 2017

referenced as prerequisite to #289

@v1k0d3n v1k0d3n changed the title feat: Update Glance Keystone Jobs to match common architecture clean: Update Glance Keystone Jobs to match common architecture Mar 18, 2017
Sign up for free to 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.

3 participants