Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

[v2] cloudstack fixes #1322

Merged
merged 1 commit into from
Dec 5, 2015
Merged

[v2] cloudstack fixes #1322

merged 1 commit into from
Dec 5, 2015

Conversation

resmo
Copy link
Contributor

@resmo resmo commented Dec 3, 2015

This PR has target devel but these fixes are meant to be for v2 as well. Let me know if you need a separate PR into stable-2.0 branch.

@gregdek
Copy link
Contributor

gregdek commented Dec 3, 2015

Thanks @resmo. Marking for inclusion.

@resmo resmo changed the title cloudstack fixes [v2] cloudstack fixes Dec 3, 2015
@bcoca
Copy link
Member

bcoca commented Dec 3, 2015

can you squash the commits?

- cs_affinitygroup: add project support
  Project support in CloudStack for affinity groups is going to be fixed/implemented in the near future, this module should already support.

- cs_affinitygroup:  fix missing returns in doc

- cs_volume: fix disk offering not found

- cs_volume: fix volume not found if created with display_volume=no

- cs_firewall: argument zone is missing, default zone is always used.
  credits for reporting and fixing to @atsaki
  closes #1320

- cs_instance: fix user_data base64 encoding fails if not a string
@resmo
Copy link
Contributor Author

resmo commented Dec 4, 2015

@bcoca squashed.

bcoca added a commit that referenced this pull request Dec 5, 2015
@bcoca bcoca merged commit 5e52b2f into ansible:devel Dec 5, 2015
@bcoca
Copy link
Member

bcoca commented Dec 5, 2015

I did not realize cs_volume is 2.1, had to remove it from the backport to stable-2.0 to be able to apply the fixes to the other modules.

@resmo
Copy link
Contributor Author

resmo commented Dec 5, 2015

oh. my fault sorry. next time I make 2 PR. one for devel and one for stable-2.0

@resmo resmo deleted the for-ansible branch December 5, 2015 07:01
@bcoca
Copy link
Member

bcoca commented Dec 5, 2015

no worries, just informing you because the commits don't look the same and the attribution might be messed up

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

Successfully merging this pull request may close these issues.

3 participants