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 get_cloud_credentials #51

Merged
merged 1 commit into from
Oct 24, 2017
Merged

Conversation

konstan
Copy link
Member

@konstan konstan commented Oct 24, 2017

Connected to #50

Copy link
Member

@0xbase12 0xbase12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why we should add such funtion in the API.

It's possible to make a function in the user code for that.

If we add such function it will be a mess in some months because there is a lot of combination with our cimi resources.

Imagine if I have added service offer related functions in the API ;).

Cal want to add a workflow abilties but it should not be directly in the API. It can be in another python module.

@konstan
Copy link
Member Author

konstan commented Oct 24, 2017

I need this for HBP MOOC and added it there because there is no option to add it anywhere else. It will then go to credential.py here
https://github.com/slipstream/SlipStreamPythonAPI/tree/issue_%2340-rtp-put-get-sse/api/src/slipstream/api
This is the branch for the migration of run->deployment that was parked. Here is the issue for the partial merge from that branch #52 This was agreed with Cal.

@0xbase12 0xbase12 merged commit 30e62b8 into master Oct 24, 2017
@0xbase12 0xbase12 deleted the issue_50-get_cloud_credentials branch October 24, 2017 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants