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

default profile broken by PR #155 #166

Closed
slapcat opened this issue Dec 21, 2023 · 1 comment · Fixed by #167
Closed

default profile broken by PR #155 #166

slapcat opened this issue Dec 21, 2023 · 1 comment · Fixed by #167
Assignees

Comments

@slapcat
Copy link
Contributor

slapcat commented Dec 21, 2023

After pulling the latest commits, I could no longer run ./configure. I believe the relevant change came in pull request #155. Rolling back to a previous version of the openstack/profiles/default file fixed the issue. Here is the output from the broken attempt to configure a Ussuri cloud:

$ ./configure
+ [[ -z 10.230.64.2 ]]
+ [[ -z 10.230.19.58 ]]
+ ./profiles/default
+ . ./profiles/common
+++ dirname ./profiles/default
++ . ./profiles/../common/juju_helpers
+++ is_juju_2
+++ juju --version
+++ egrep -q '^2.'
+++ JUJU_RUN_CMD='run-action --wait'
+ install_packages
+ echo 'NOTE: The configure script no longer sudo apt installs packages. If you find a client is missing on your machine, resolve it manually in advance of ./configure.'
NOTE: The configure script no longer sudo apt installs packages. If you find a client is missing on your machine, resolve it manually in advance of ./configure.
+ [[ -z 10.230.64.2 ]]
+ [[ -z 10.230.19.58 ]]
+ [[ -z 10.5.0.1 ]]
+ [[ -z 10.5.0.0/16 ]]
+ [[ -z 10.5.150.0:10.5.200.254 ]]
+ [[ -z 192.168.21.0/24 ]]
+ [[ -z admin_admin_net ]]
+ NS_ARG=
++ juju config neutron-api enable-ml2-dns
+ '[' true = true ']'
+ unset NAMESERVER
++ juju status ovn-chassis --format=json
++ jq '.machines| length'
+ ovn_units=1
+ (( ovn_units ))
+ ovn_enabled=true
+ true
+ net_type=geneve
++ juju config neutron-api enable-dvr
+ dvr_enabled=false
+ '[' false = true ']'
+ true
+ false
+ ./bin/add-data-ports.sh '' admin_admin_net
Managing ports for ovn-chassis units
Checking ovn-chassis unit instances: 22702e65-b8da-4908-8587-e5fbf7057756
No Network found for admin_admin_net
No Network found for admin_admin_net
Creating 1 new ports
No Network found for admin_admin_net
@nicolasbock
Copy link
Collaborator

Nice catch and forensics @slapcat !

I'll prepare a fix here momentarily....

@nicolasbock nicolasbock self-assigned this Dec 21, 2023
nicolasbock added a commit to nicolasbock/stsstack-bundles that referenced this issue Dec 21, 2023
nicolasbock added a commit to nicolasbock/stsstack-bundles that referenced this issue Dec 22, 2023
nicolasbock added a commit to nicolasbock/stsstack-bundles that referenced this issue Dec 22, 2023
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 a pull request may close this issue.

2 participants