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.

Sign up for GitHub

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

Jump to bottom

linchpin rise should not require inventory layout #154

Closed
herlo opened this issue Feb 16, 2017 · 0 comments
Closed

linchpin rise should not require inventory layout #154

herlo opened this issue Feb 16, 2017 · 0 comments
Assignees
@herlo
Labels
bug
Projects
Release v0.9.x - Inception -> 20170404
Milestone
v0.9.0

Comments

@herlo
Copy link
Contributor

herlo commented Feb 16, 2017

When creating a PinFile, one should be able to list a Pin target without a declared layout.

Currently the following fails:

---
libvirt-test:
  topology: simple-libvirt-cluster.yml

But

---
libvirt-test:
  topology: simple-libvirt-cluster.yml
  layout: openshift-3node-cluster.yml

succeeds.

The former needs to be allowed. In many cases, a resulting inventory is not desired.
@herlo herlo added the bug label Feb 16, 2017
@herlo herlo self-assigned this Feb 16, 2017
@herlo herlo added this to the v0.9.0 milestone Feb 16, 2017
herlo added a commit to herlo/linchpin that referenced this issue Feb 16, 2017
@herlo
fixes issue CentOS-PaaS-SIG#154
5b877ba 
Remove the requirement for the layout of a particular PinFile target
samvarankashyap pushed a commit to samvarankashyap/linchpin that referenced this issue Feb 20, 2017
@herlo @samvarankashyap
fixes issue CentOS-PaaS-SIG#154
e9f5d5a 
Remove the requirement for the layout of a particular PinFile target
samvarankashyap pushed a commit to samvarankashyap/linchpin that referenced this issue Mar 9, 2017
@herlo
fixes issue CentOS-PaaS-SIG#154
eb27511 
Remove the requirement for the layout of a particular PinFile target
@adl-bot adl-bot mentioned this issue May 31, 2018
Merged
@adl-bot adl-bot mentioned this issue Aug 17, 2018
Merged
@adl-bot adl-bot mentioned this issue Sep 9, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@herlo herlo

Labels
bug
Projects
No open projects
Release v0.9.x - Inception -> 20170404
Completed
Milestone
v0.9.0
Development

No branches or pull requests
2 participants
@herlo @samvarankashyap