Skip to content

Conversation

l3acon
Copy link
Contributor

@l3acon l3acon commented Mar 6, 2025

What does this PR do?

When running a fairly straight-forward playbook (as alluded to in the docs), I get the following failures:

  • valid_tags is undefined
  • template errors when running against AAP 2.5
  • variable mis-match from controller_hostname ...etc to aap_hostname...etc

Of note, I still get failures when attempting to export Applications, I didn't need this functionality but perhaps we should look into fixing downstream resources exports.

How should this be tested?

Include filetre_create role.

Is there a relevant Issue open for this?

No that I'm aware of, but as mentioned above filetree_create role didn't work for me without these changes.

Copy link
Contributor

@ivarmu ivarmu left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your PR and your effort, I appreciate it a lot. That being said, there's missing some information, as what errors do you get without these changes, what versions are you using...

Please check the individual review comments.

Copy link
Contributor

Choose a reason for hiding this comment

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

We are not going back with these names. For AAP2.5, aap_* are the correct ones and are aligned with the main collection:

https://github.com/redhat-cop/infra.aap_configuration/blob/devel/roles%2Fcontroller_organizations%2Ftasks%2Fmain.yml#L18-L24

So, please adapt your input variables to the new names for this collection.

@@ -50,7 +50,6 @@
args:
apply:
tags: "{{ input_tag | to_yaml }}"
tags: "{{ valid_tags }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the matter with this? This is required to run correctly when you only need to export one or more objet types, but not all of them.

lennysh and others added 4 commits March 25, 2025 15:36
2. Add double quotes for `*_nodes` lines that may contain special characters (like #)
3. Fix Truthy value for `survey_enabled` coming in as `True` or `False` instead of `true` or `false`
* misc: inventory name unification

* fix: missing change

* fix: missing change after merge
@l3acon l3acon requested a review from a team as a code owner March 25, 2025 21:36
@l3acon
Copy link
Contributor Author

l3acon commented Mar 25, 2025

@ivarmu apologies, I was too hasty with this. I have removed every change that is not required to get this functional in my environment. I have also created the issue this addresses #89

Copy link
Contributor

@ivarmu ivarmu left a comment

Choose a reason for hiding this comment

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

LGTM. It adds an extra check to be sure there always will be at least one item in the list. Good job!

@ivarmu ivarmu merged commit 4246791 into redhat-cop:devel Mar 31, 2025
9 checks passed
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.

4 participants