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

[PR #2028/bfd1d80b backport][stable-7] Fix example of eks_nodegroup #2035

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 4, 2024

This is a backport of PR #2028 as merged into main (bfd1d80).

SUMMARY

When using the example as a reference, ansible-playbook command outputs the following errors:

fatal: [localhost]: FAILED! => {"changed": false, "msg": "argument 'scaling_config' is of type <class 'list'> and we were unable to convert to dict: <class 'list'> cannot be converted to a dict"}

fatal: [localhost]: FAILED! => {"changed": false, "msg": "argument 'labels' is of type <class 'list'> and we were unable to convert to dict: <class 'list'> cannot be converted to a dict"}

fatal: [localhost]: FAILED! => {"changed": false, "msg": "value of capacity_type must be one of: ON_DEMAND, SPOT, got: on_demand"}

This fixes those issues.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

eks_nodegroup

ADDITIONAL INFORMATION

Fix example of eks_nodegroup

SUMMARY
When using the example as a reference, ansible-playbook command outputs the following errors:
fatal: [localhost]: FAILED! => {"changed": false, "msg": "argument 'scaling_config' is of type <class 'list'> and we were unable to convert to dict: <class 'list'> cannot be converted to a dict"}

fatal: [localhost]: FAILED! => {"changed": false, "msg": "argument 'labels' is of type <class 'list'> and we were unable to convert to dict: <class 'list'> cannot be converted to a dict"}

fatal: [localhost]: FAILED! => {"changed": false, "msg": "value of capacity_type must be one of: ON_DEMAND, SPOT, got: on_demand"}

This fixes those issues.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
eks_nodegroup
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
(cherry picked from commit bfd1d80)
@patchback patchback bot mentioned this pull request Jan 4, 2024
Copy link

github-actions bot commented Jan 4, 2024

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/f295b2e1898040ef886c6cdfee85aedf

ansible-galaxy-importer FAILURE in 5m 10s (non-voting)
✔️ build-ansible-collection SUCCESS in 15m 34s
✔️ ansible-test-splitter SUCCESS in 5m 50s
✔️ integration-community.aws-1 SUCCESS in 28m 14s
Skipped 21 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jan 5, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/25d651c04012497d99dc1e4dfb60dfab

ansible-galaxy-importer FAILURE in 4m 41s (non-voting)
✔️ build-ansible-collection SUCCESS in 15m 08s
✔️ ansible-test-splitter SUCCESS in 6m 12s
✔️ integration-community.aws-1 SUCCESS in 29m 55s
Skipped 21 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit f96dba0 into stable-7 Jan 5, 2024
69 of 73 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-7/bfd1d80b6aae496a97e4e0f7fcee2a8f9438116e/pr-2028 branch January 5, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants