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

Fixes #7248,BZ1132676 - Calling the correct resource method #13

Merged
merged 1 commit into from
Aug 27, 2014

Conversation

daviddavis
Copy link

No description provided.

@daviddavis daviddavis changed the title [DO NOT REVIEW] Finding the resource instead of changing it Fixes #7248,BZ1132676 - Calling the correct resource method Aug 25, 2014
@bbuckingham
Copy link
Member

Will test this out

@bbuckingham
Copy link
Member

ACK. Tested this out with some capsule syncs and it works perfectly.

With the change:

hammer> capsule content synchronize --id 3 --environment-id 5
[..................................................................................................................................] [100%]
Task 1cc799db-7db9-4b04-8443-12fb9ebb7b7e: success
hammer> capsule content synchronize --id 3 --environment-id 5
[..................................................................................................................................] [100%]
Task 740eb45a-4c69-4e43-a3c1-0548860c3e9e: success
hammer> capsule content synchronize --id 3 --environment-id 5
[..................................................................................................................................] [100%]
Task 0d0c2bcc-4d08-43a9-abde-4e56e16890fc: success

Without the change:

hammer> capsule content synchronize --id 3 --environment-id 5
[..................................................................................................................................] [100%]
Task 6c86b093-45c4-4d2b-999d-630ed17b31ff: success
hammer> capsule content synchronize --id 3 --environment-id 5
Could not synchronize capsule content:
  Error: The server does not support such operation.

iNecas added a commit that referenced this pull request Aug 27, 2014
Fixes #7248,BZ1132676 - Calling the correct resource method
@iNecas iNecas merged commit f59a115 into theforeman:master Aug 27, 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

3 participants