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

Fix for ruamel interactions with awxkit objects #338

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

JacobCallahan
Copy link
Member

Cole encountered an issue where inventory syncs were failing because ruamel wasn't able to handle awxkit's Psuedonamespace objects. This simply adds a new representer for them, fixing the issue.

Cole encountered an issue where inventory syncs were failing because
ruamel wasn't able to handle awxkit's Psuedonamespace objects. This
simply adds a new representer for them, fixing the issue.
@JacobCallahan JacobCallahan added the bug Something isn't working label Nov 26, 2024
@JacobCallahan JacobCallahan self-assigned this Nov 26, 2024

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.

Comments skipped due to low confidence (1)

broker/providers/ansible_tower.py:32

  • The word 'PsuedoNamespace' is misspelled. It should be 'PseudoNamespace'.
"""Recursively convert PsuedoNamespace objects into dictionaries."""
@JacobCallahan JacobCallahan marked this pull request as ready for review November 26, 2024 18:49
Copy link
Member

@ColeHiggins2 ColeHiggins2 left a comment

Choose a reason for hiding this comment

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

Tested locally and works great!

@JacobCallahan JacobCallahan merged commit 5eff114 into SatelliteQE:master Nov 27, 2024
4 checks passed
@JacobCallahan JacobCallahan deleted the ruamel_fix branch November 27, 2024 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants