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

rest-api: Update background operations #108

Merged
merged 2 commits into from
Dec 4, 2014
Merged

rest-api: Update background operations #108

merged 2 commits into from
Dec 4, 2014

Conversation

stgraber
Copy link
Contributor

@stgraber stgraber commented Dec 2, 2014

Return the metadata dict on creation and allow storing multiple related
resources.

Signed-off-by: Stéphane Graber stgraber@ubuntu.com

Return the metadata dict on creation and allow storing multiple related
resources.

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
'resource': "/1.0/containers/my-container" # Affected resource
'operation': "/1.0/containers/<id>", # URL to the background operation
'resources': {
'containers': [["/1.0/containers/my-container"] # List of affected resources
Copy link
Contributor

Choose a reason for hiding this comment

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

The [[ here looks like a typo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed

 - Fix broken JSON
 - Drop result and result_code from operation, use status instead
 - Add timeout arg to operation /wait

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
tych0 added a commit that referenced this pull request Dec 4, 2014
rest-api: Update background operations
@tych0 tych0 merged commit 16765fa into canonical:master Dec 4, 2014
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.

None yet

2 participants