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

fix #46187: Adds injected_file_content_bytes and injected_file_path_b… #52955

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
3 participants
@userlerueda
Copy link
Contributor

userlerueda commented Feb 25, 2019

…ytes

SUMMARY

Adds injected_file_content_bytes and injected_file_path_bytes in order to fix the problem described in #46187

Fixes #46187

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

os_quota.py

ADDITIONAL INFORMATION

Tested using the new options

$ ansible-playbook test/develop/set_quotas.yaml

PLAY [Test os_quotas] ****************************************************************************************************************************************************************************************************

TASK [Gathering Facts] ***************************************************************************************************************************************************************************************************
ok: [localhost]

TASK [Get quotas before changing] ****************************************************************************************************************************************************************************************
ok: [localhost]

TASK [debug] *************************************************************************************************************************************************************************************************************
ok: [localhost] => {
    "before.openstack_quotas.compute.injected_file_content_bytes": "65536"
}

TASK [Set quotas] ********************************************************************************************************************************************************************************************************
changed: [localhost]

TASK [Get quotas after changing] *****************************************************************************************************************************************************************************************
ok: [localhost]

TASK [debug] *************************************************************************************************************************************************************************************************************
ok: [localhost] => {
    "after.openstack_quotas.compute.injected_file_content_bytes": "888"
}

PLAY RECAP ***************************************************************************************************************************************************************************************************************
localhost                  : ok=6    changed=1    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0
@openstack-zuul

This comment has been minimized.

Copy link

openstack-zuul bot commented Feb 25, 2019

Build succeeded (third-party-check pipeline).

@mnaser

This comment has been minimized.

Copy link
Contributor

mnaser commented Feb 26, 2019

shipit

@ansibot ansibot removed the needs_triage label Feb 26, 2019

@userlerueda

This comment has been minimized.

Copy link
Contributor Author

userlerueda commented Feb 28, 2019

bot_status

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Mar 5, 2019

Components

lib/ansible/modules/cloud/openstack/os_quota.py
support: community
maintainers: Shrews cloudnull dagnello emonty evrardjp juliakreger mgale mnaser odyssey4me rcarrillocruz

Metadata

waiting_on: maintainer
changes_requested_by: null
needs_info: False
needs_revision: False
needs_rebase: False
merge_commits: []
too many files or commits: False
mergeable_state: clean
shippable_status: success
maintainer_shipits (module maintainers): 1
community_shipits (namespace maintainers): 0
ansible_shipits (core team members): 0
shipit_actors (maintainers or core team members): mnaser
shipit_actors_other: []
automerge: automerge shipit test failed

click here for bot help

@userlerueda

This comment has been minimized.

Copy link
Contributor Author

userlerueda commented Mar 6, 2019

Can we get another shipit ?

@ansibot ansibot added the stale_ci label Mar 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.