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

Feat ansible playbook: Added Ansible Playbook Conversion step #262

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

vanisrikanithi
Copy link
Contributor

Ansible Playbook go-conversion:
Added "Ansible Playbook " support to go-convert specific to jenkins migration.

Ansible Playbook Jenkins Trace:
vani_ansible.json

go run main.go jenkinsjson --downgrade ~/go/vani_ansible.json

Harness Test Pipeline here.

Ansible Playbook step:

- step:
    identifier: ansibleplaybookb2fcf1
    name: Ansible_Playbook
    spec:
      image: plugins/ansible
      settings:
        become: "true"
        become_user: root
        check: "false"
        extra_vars: var2=value2,var1=value1
        forks: "5"
        inventory: inventory.ini
        limit: localhost
        list_hosts: ""
        playbook: example-playbook.yml
        skip_tags: debug
        start_at_task: ""
        tags: setup,deploy
        vault_id: ""
        vault_password: <secrets.getValue('vault-password')>
    timeout: ""
    type: Plugin

@vanisrikanithi
Copy link
Contributor Author

@Ompragash, Added a Ansible Playbook Conversion step. Could you please check and provide your comments on this PR?

"start_at_task": startAtTask,
"tags": tags,
"vault_id": vaultCredentialsId,
"vault_password": "<secrets.getValue('vault-password')>",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vault_password is optional and should be called only when seen in the trace - this can go under if statement

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.

2 participants