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

Can we remove iaas_configuration_guid from staged-director-config? #190

Closed
mcwumbly opened this issue Jun 15, 2018 · 3 comments
Closed

Can we remove iaas_configuration_guid from staged-director-config? #190

mcwumbly opened this issue Jun 15, 2018 · 3 comments

Comments

@mcwumbly
Copy link
Contributor

In #182, we added back iaas_configuration_guid to the output of staged-director-config under the assumption that it will be necessary to support the new multi-CPI features in OM. But now we are wondering if that is really necessary. Could we instead fetch this information on demand the same way we are doing for networks and availability zones?

We need to verify with the OM team what is safe to assume. For example, can you have availability zones with the same name on two different iaas_configurations? If not, we may be able to remove these guids. If so, we likely have to keep them...

Use reference:
http://docs.pivotal.io/pivotalcf/2-2/opsman-api/#fetching-iaas-configurations

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@kcboyle
Copy link
Contributor

kcboyle commented Jun 4, 2019

This appears to cause issues when using the top-level key: iaas-configurations.

if using staged-director-config, we cannot set configure-director if using iaas-configurations due to this issue.

@nbconklin
Copy link
Contributor

@mcwumbly I'm going to close this conversation as we have #377 now. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants