Skip to content

Updated YAML Example Playbook#253

Open
Siddharthablog wants to merge 1 commit intoansible-middleware:mainfrom
Siddharthablog:main
Open

Updated YAML Example Playbook#253
Siddharthablog wants to merge 1 commit intoansible-middleware:mainfrom
Siddharthablog:main

Conversation

@Siddharthablog
Copy link
Copy Markdown

Changes

  • Added a comprehensive example playbook with common and advanced variables for ActiveMQ uninstall.

  • Clarified usage of the roles: section and provided correct playbook formatting for role invocation.

@RanabirChakraborty
Copy link
Copy Markdown
Member

@Siddharthablog
Copy link
Copy Markdown
Author

@RanabirChakraborty
Copy link
Copy Markdown
Member

@Siddharthablog make the 2 commit as one , as it's a single issue only.

Updated YAML for activemq Example Playbook
Updated YAML for activemq uninstall Example Playbook
@Siddharthablog
Copy link
Copy Markdown
Author

@RanabirChakraborty Done!

@RanabirChakraborty
Copy link
Copy Markdown
Member

@Siddharthablog please see the earlier comments and fix accordingly.

@hcherukuri
Copy link
Copy Markdown
Member

@Siddharthablog Are you looking into this?

@Siddharthablog
Copy link
Copy Markdown
Author

@hcherukuri I am unable to understand where exactly change is required, as per my understanding I have already updated the YAML file. So please you can look into it. Thanks


roles:
- middleware_automation.amq.activemq_uninstall
- role: middleware_automation.amq.activemq_uninstall
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

remove "-role" keep it the way it was before.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Siddharthablog looks like it's not done.

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.

3 participants