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

CI: Enable openstack-cli-server #205

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mandre
Copy link
Collaborator

@mandre mandre commented Jan 30, 2025

Similar to what was done in gophercloud/gophercloud@f018747

This allows to save a couple of minutes standing up devstack.

@github-actions github-actions bot added the semver:patch No API change label Jan 30, 2025
@mandre
Copy link
Collaborator Author

mandre commented Jan 30, 2025

Weird, the network controller is waiting for provider_net_dualstack_1 to be created externally, while we should have done it as part of e2e.sh script earlier.

@mandre mandre force-pushed the enable-openstack-client-server branch from e9e80fa to 4f23daa Compare January 30, 2025 13:08
@mdbooth mdbooth enabled auto-merge January 30, 2025 13:29
@mdbooth mdbooth disabled auto-merge January 30, 2025 13:31
@mdbooth
Copy link
Contributor

mdbooth commented Jan 30, 2025

I was going to let this auto-merge, but we should probably manually check that the tests did what we wanted before merging, even if they pass.

@github-actions github-actions bot added semver:major Breaking change and removed semver:patch No API change labels Jan 30, 2025
@mdbooth mdbooth removed the semver:major Breaking change label Jan 30, 2025
@mdbooth mdbooth force-pushed the enable-openstack-client-server branch from 0e3ef5a to 04ab929 Compare January 30, 2025 19:06
@github-actions github-actions bot added the semver:patch No API change label Jan 30, 2025
@mdbooth mdbooth force-pushed the enable-openstack-client-server branch 2 times, most recently from 3f7c088 to 5b0f1fc Compare January 30, 2025 20:29
@mdbooth
Copy link
Contributor

mdbooth commented Jan 30, 2025

There's something very weird going on here. The private network seems to entirely disappear after being renamed. #212 might allow us to work round it, while also being generally more flexible.

@mandre mandre changed the title CI: Enable openstack-client-server CI: Enable openstack-cli-server Jan 31, 2025
@mandre
Copy link
Collaborator Author

mandre commented Jan 31, 2025

Really funky stuff going on there 🤔

@mdbooth mdbooth force-pushed the enable-openstack-client-server branch 2 times, most recently from 0ffb4bb to 8dda588 Compare January 31, 2025 11:22
mandre and others added 2 commits January 31, 2025 12:51
Similar to what was done in gophercloud/gophercloud@f018747

This allows to save a couple of minutes standing up devstack.
@mdbooth mdbooth force-pushed the enable-openstack-client-server branch from 8dda588 to 3af240d Compare January 31, 2025 13:01
@mandre
Copy link
Collaborator Author

mandre commented Feb 3, 2025

From the devstack logs, the last thing we see for the private network is:

Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.maintenance [None req-f60b0267-d3e5-4bc7-8f5a-359ebbc1de70 None None] Maintenance task: Synchronizing Neutron and OVN databases started {{(pid=61700) check_for_inconsistencies /opt/stack/neutron/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py:355}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.maintenance [None req-f60b0267-d3e5-4bc7-8f5a-359ebbc1de70 None None] Maintenance task: Number of inconsistencies found at create/update: networks=2 {{(pid=61700) _log /opt/stack/neutron/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py:336}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.maintenance [None req-f60b0267-d3e5-4bc7-8f5a-359ebbc1de70 None None] Maintenance task: Fixing resource 32709dd5-272a-4264-b397-8b610b589514 (type: networks) at create/update {{(pid=61700) check_for_inconsistencies /opt/stack/neutron/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py:365}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61696]: INFO neutron.api.wsgi [None req-7cbcf174-0161-4c46-be44-7aaf1265b3ae demo demo] 172.18.0.2 "GET /networking/v2.0/networks?name=private HTTP/1.1" status: 200  len: 189 time: 0.0070784
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG ovsdbapp.backend.ovs_idl.transaction [-] Running txn n=1 command(idx=0): CheckRevisionNumberCommand(_result=None, name=neutron-32709dd5-272a-4264-b397-8b610b589514, resource={'id': '32709dd5-272a-4264-b397-8b610b589514', 'name': 'private', 'tenant_id': 'bdc69105818e425ba90ee7dbc8c30d04', 'admin_state_up': True, 'mtu': 1442, 'status': 'ACTIVE', 'subnets': ['0df9e003-9d40-413c-9c79-64afb20986dc', '77ce5222-c90c-4695-8c2c-55b3afab71bd'], 'standard_attr_id': 12, 'shared': False, 'availability_zone_hints': [], 'availability_zones': [], 'ipv4_address_scope': None, 'ipv6_address_scope': None, 'router:external': False, 'vlan_transparent': None, 'description': '', 'port_security_enabled': True, 'tags': [], 'created_at': '2025-01-31T13:10:02Z', 'updated_at': '2025-01-31T13:10:03Z', 'revision_number': 3, 'project_id': 'bdc69105818e425ba90ee7dbc8c30d04', 'provider:network_type': 'geneve', 'provider:physical_network': None, 'provider:segmentation_id': 15486}, resource_type=networks, if_exists=True) {{(pid=61700) do_commit /opt/stack/data/venv/lib/python3.10/site-packages/ovsdbapp/backend/ovs_idl/transaction.py:89}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG ovsdbapp.backend.ovs_idl.transaction [-] Running txn n=1 command(idx=1): DbSetCommand(_result=None, table=Logical_Switch, record=neutron-32709dd5-272a-4264-b397-8b610b589514, col_values=(('external_ids', {'neutron:network_name': 'private', 'neutron:mtu': '1442', 'neutron:revision_number': '3', 'neutron:availability_zone_hints': '', 'neutron:provnet-network-type': 'geneve'}), ('other_config', {'mcast_snoop': 'false', 'mcast_flood_unregistered': 'false', 'vlan-passthru': 'false'})), if_exists=True) {{(pid=61700) do_commit /opt/stack/data/venv/lib/python3.10/site-packages/ovsdbapp/backend/ovs_idl/transaction.py:89}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: DEBUG ovsdbapp.backend.ovs_idl.transaction [-] Running txn n=1 command(idx=2): SetLSwitchPortCommand(_result=None, lport=provnet-06d58dc2-47bf-4e93-8cfe-9682a943e935, external_ids_update=None, columns={'tag': 15486}, if_exists=True) {{(pid=61700) do_commit /opt/stack/data/venv/lib/python3.10/site-packages/ovsdbapp/backend/ovs_idl/transaction.py:89}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61697]: DEBUG neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.ovsdb_monitor [None req-5c408af6-16f6-428a-b1ff-83bfedf12722 None None] Hash Ring: Node 6ac6980c-3e2a-466b-9036-a0802f378832 (host: fv-az1456-811) handling event "update" for row 7b86d717-14a6-4546-b9cf-9d6a16f23c62 (table: Logical_Switch) {{(pid=61697) notify /opt/stack/neutron/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py:855}}
Jan 31 13:14:58 fv-az1456-811 neutron-server[61700]: INFO neutron.db.ovn_revision_numbers_db [None req-f60b0267-d3e5-4bc7-8f5a-359ebbc1de70 None None] Successfully bumped revision number for resource 32709dd5-272a-4264-b397-8b610b589514 (type: networks) to 3

Then further GET for the private network seemingly return no results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch No API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants