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 ovn-ic #86

Merged

Conversation

LorenzoBianconi
Copy link
Contributor

No description provided.

@LorenzoBianconi LorenzoBianconi force-pushed the ovn-ic-central-nodes-rework branch from 9deb937 to 44cb267 Compare November 1, 2023 09:21
@LorenzoBianconi LorenzoBianconi changed the title Ovn ic central nodes rework Fix for ovn-ic Nov 1, 2023
@numansiddique
Copy link
Contributor

@LorenzoBianconi Signed-off-by tag please

@LorenzoBianconi LorenzoBianconi force-pushed the ovn-ic-central-nodes-rework branch from 44cb267 to 4835365 Compare November 27, 2024 14:15
@LorenzoBianconi
Copy link
Contributor Author

@LorenzoBianconi Signed-off-by tag please

@numansiddique done

@numansiddique
Copy link
Contributor

@LorenzoBianconi Cn you plz rebase and trigger another CI run ?

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
@LorenzoBianconi LorenzoBianconi force-pushed the ovn-ic-central-nodes-rework branch from 4835365 to d6b36b0 Compare November 28, 2024 15:51
@numansiddique numansiddique merged commit 75086e2 into ovn-org:main Nov 28, 2024
2 checks passed
@igsilya
Copy link
Contributor

igsilya commented Nov 30, 2024

@numansiddique @LorenzoBianconi This change seems to have broken ovn-heater for the case with OVN-IC with standalone databases. In this case the CLUSTER_IC_ID is defined as ovn-central-az0 <-- see that there is no -1 suffix and so it fails to find the container:

TASK [Start central containers] ************************************************
failed: [server03] (item=0) => {
  "ansible_loop_var": "item",
  "changed": true,
  "cmd": "./ovn_cluster.sh start\n",
  "delta": "0:00:03.577927",
  "end": "2024-11-30 02:01:20.587213",
  "item": 0,
  "msg": "non-zero return code",
  "rc": 125,
  "start": "2024-11-30 02:01:17.009286",
  "stderr": "Error: no container with name or ID \"ovn-central-az0\" found: no such container",
  "stderr_lines": [
    "Error: no container with name or ID \"ovn-central-az0\" found: no such container"
  ],
  "stdout": "Starting OVN cluster\nWaiting for containers to be up..\nAdding ovs-ports\n/etc/ovn/ovnnb_db.db does not exist ... (warning).\nCreating empty database /etc/ovn/ovnnb_db.db [  OK  ]\r\nStarting ovsdb-nb [  OK  ]\r\n/etc/ovn/ovnsb_db.db does not exist ... (warning).\nCreating empty database /etc/ovn/ovnsb_db.db [  OK  ]\r\nStarting ovsdb-sb [  OK  ]\r\nStarting ovn-northd [  OK  ]",
  "stdout_lines": [
    "Starting OVN cluster",
    "Waiting for containers to be up..",
    "Adding ovs-ports",
    "/etc/ovn/ovnnb_db.db does not exist ... (warning).",
    "Creating empty database /etc/ovn/ovnnb_db.db [  OK  ]",
    "Starting ovsdb-nb [  OK  ]",
    "/etc/ovn/ovnsb_db.db does not exist ... (warning).",
    "Creating empty database /etc/ovn/ovnsb_db.db [  OK  ]",
    "Starting ovsdb-sb [  OK  ]",
    "Starting ovn-northd [  OK  ]"
  ]
}

The suffix must be added only for the clustered case. At least, the users do not expect to have to add the suffix.
Could you, please, take a look?

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