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

Add more features for openstack orchestration #5

Closed
wants to merge 1 commit into from

Conversation

bzwei
Copy link

@bzwei bzwei commented Sep 10, 2014

The following methods are added to Fog::Orchestration::OpenStack

  1. describe_stack - get more details of a stack, including parameters, output etc.
  2. list_stack_resources - get a list of all resources generated by the stack
  3. get_template - get the template used to generate the stack

The following methods are added to Fog::Orchestration::OpenStack
1. describe_stack - get more details of a stack, including parameters, output etc.
2. list_stack_resources - get a list of all resources generated by the stack
3. get_template - get the template used to generate the stack
@kbrock
Copy link
Member

kbrock commented Oct 13, 2014

Spoke with Bill.

Looks like there is an outstanding datamodel request at fog/fog-core#45.
There is also an outstanding PR that leverages these changes. It does not support open stack yet, but the author does share that there is one in the works: fog#2971

If nothing else, we want to rebase to recent master.

@Fryguy
Copy link
Member

Fryguy commented Dec 9, 2014

@bzwei Is this required for the refresh code?

@bzwei
Copy link
Author

bzwei commented Dec 9, 2014

Yes. This is needed for openstack refresh.

@jrafanie
Copy link
Member

jrafanie commented Dec 9, 2014

@bzwei @kbrock is this implementation the same that's in those fog pull requests? I don't see the same commits here so I'm not sure. It's tricky to implement our own fork of the code. I would rather help get the right code merged upstream and then use that version.

@bzwei
Copy link
Author

bzwei commented Dec 9, 2014

The fog community was refactoring the related code but halted, those PRs were never merged. The changes were too big and I'm afraid we can help to make the refactoring completed. My implementation here was to add a few methods to their existing code with the old style.

@jrafanie
Copy link
Member

jrafanie commented Dec 9, 2014

@bzwei from the link to fog's issue 45 from @kbrock's comment, it looks like they're either waiting for us or it's going to be implemented in a separate library. Let's not fork fog until we have exhausted all options. Maintaining forks for libraries that we often need to rebase from upstream changes is a huge technical debt.

@bzwei
Copy link
Author

bzwei commented Feb 10, 2015

The PR is now abandoned. The new implementation fog#3412 has been merged to fog/fog

@bzwei bzwei closed this Feb 10, 2015
@bzwei bzwei deleted the orchestration branch February 10, 2015 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants