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

Invalid module options when using role content_view_publish #1698

Closed
lucas-benedito opened this issue Jan 8, 2024 · 3 comments · Fixed by #1701
Closed

Invalid module options when using role content_view_publish #1698

lucas-benedito opened this issue Jan 8, 2024 · 3 comments · Fixed by #1701
Assignees

Comments

@lucas-benedito
Copy link

SUMMARY

After implementation from #1676, the role results in a failure due to invalid parameters

ISSUE TYPE
  • Bug Report
ANSIBLE VERSION
ansible [core 2.15.1]
COLLECTION VERSION
theforeman.foreman             3.15.0
STEPS TO REPRODUCE
---
- name: Testing async settings
  hosts: localhost
  tasks:
    - ansible.builtin.include_role:
        name: theforeman.foreman.content_view_publish
      vars:
        foreman_content_view_publish_async: 7200
        foreman_content_view_publish_poll: 30
        foreman_server_url: https://testhost
        foreman_organization: "test"
        foreman_username: "admin"
        foreman_password: test
        foreman_content_views:
          - "test"
EXPECTED RESULTS

Playbook executed with async and poll settings

ACTUAL RESULTS
TASK [theforeman.foreman.content_view_publish : Publish content views] *********************************************************************************************************************************************************************************************************
failed: [localhost] (item=test) => {"ansible_loop_var": "content_view", "changed": false, "content_view": "test", "msg": "Unsupported parameters for (theforeman.foreman.content_view_version) module: async, poll. Supported parameters include: content_view, current_lifecycle_environment, description, force_promote, force_yum_metadata_regeneration, lifecycle_environments, organization, password, server_url, state, username, validate_certs, version (force)."}
@lucas-benedito
Copy link
Author

lucas-benedito commented Jan 10, 2024

@Griffin-Sullivan from the tests performed, this was already in place as per #1676
The problem, however, is that the module content_view_version does not accept the parameters async and poll

@evgeni
Copy link
Member

evgeni commented Jan 10, 2024

Correct, those are Ansible params, not module params. Griffin's patch changes that accordingly.

@lucas-benedito
Copy link
Author

I missed the tab change in your PR. Thanks for the clarification.

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 a pull request may close this issue.

3 participants