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

Bump default OpenStack version in terraform to yoga #1294

Merged
merged 1 commit into from
Sep 7, 2022
Merged

Conversation

osfrickler
Copy link
Member

Signed-off-by: Dr. Jens Harbott [email protected]

@osfrickler osfrickler marked this pull request as draft August 16, 2022 18:39
@osfrickler
Copy link
Member Author

recheck

Base automatically changed from zuul03 to main August 26, 2022 12:29
@berendt
Copy link
Member

berendt commented Sep 4, 2022

recheck

@osfrickler
Copy link
Member Author

This patch still deploys xena in CI, needs more changes.

@berendt
Copy link
Member

berendt commented Sep 4, 2022

recheck

1 similar comment
@berendt
Copy link
Member

berendt commented Sep 5, 2022

recheck

@osfrickler
Copy link
Member Author

Hitting the real issue now in CI, at least one of them:

2022-09-05 22:06:26.729377 | orchestrator | fatal: [testbed-node-0.testbed.osism.xyz]: FAILED! => {"msg": "The conditional check 'inventory_hostname in groups[item.value.group]' failed. The error was: error while evaluating conditional (inventory_hostname in groups[item.value.group]): 'dict object' has no attribute 'ironic-tftp'\n\nThe error appears to be in '/ansible/roles/ironic/tasks/config.yml': line 2, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n---\n- name: Ensuring config directories exist\n  ^ here\n"}

Will update.

@berendt
Copy link
Member

berendt commented Sep 6, 2022

@osfrickler We should add a script that sync the inventory in cfg-generics with the upstream. This way it is probably easier to identify missing inventory groups.

@osfrickler
Copy link
Member Author

recheck

@osfrickler osfrickler marked this pull request as ready for review September 7, 2022 06:50
Copy link
Contributor

@ra-beer ra-beer left a comment

Choose a reason for hiding this comment

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

LGTM

@ra-beer ra-beer merged commit 21ef674 into main Sep 7, 2022
@ra-beer ra-beer deleted the bump-yoga branch September 7, 2022 06:55
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