Skip to content

mgr/cephadm: ingress: fix typo in spec.virtual_interface_networks reference#42389

Merged
tchaikov merged 1 commit intoceph:masterfrom
asannes:ingress-fix
Jul 28, 2021
Merged

mgr/cephadm: ingress: fix typo in spec.virtual_interface_networks reference#42389
tchaikov merged 1 commit intoceph:masterfrom
asannes:ingress-fix

Conversation

@asannes
Copy link
Contributor

@asannes asannes commented Jul 18, 2021

When using virtual_inteface_networks to identify the interface to have the
virtual ip on, it referenced spec.networks instead of
spec.virtual_interface_networks.

Fixes: https://tracker.ceph.com/issues/51721
Signed-off-by: Asbjørn Sannes asbjorn.sannes@interhost.no

…erence

When using virtual_inteface_networks to identify the interface to have the
virtual ip on, it referenced spec.networks instead of
spec.virtual_interface_networks.

Fixes: https://tracker.ceph.com/issues/51721
Signed-off-by: Asbjørn Sannes <asbjorn.sannes@interhost.no>
Copy link
Member

@liewegas liewegas left a comment

Choose a reason for hiding this comment

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

Good catch! Thanks

@sebastian-philipp
Copy link
Contributor

h[11467]: Traceback (most recent call last):
h[11467]:   File "/usr/share/ceph/mgr/cephadm/serve.py", line 588, in _apply_all_services
h[11467]:     if self._apply_service(spec):
h[11467]:   File "/usr/share/ceph/mgr/cephadm/serve.py", line 811, in _apply_service
h[11467]:     daemon_spec = svc.prepare_create(daemon_spec)
h[11467]:   File "/usr/share/ceph/mgr/cephadm/services/ingress.py", line 31, in prepare_create
h[11467]:     return self.keepalived_prepare_create(daemon_spec)
h[11467]:   File "/usr/share/ceph/mgr/cephadm/services/ingress.py", line 161, in keepalived_prepare_create
h[11467]:     daemon_spec.final_config, daemon_spec.deps = self.keepalived_generate_config(daemon_spec)
h[11467]:   File "/usr/share/ceph/mgr/cephadm/services/ingress.py", line 228, in keepalived_generate_config
h[11467]:     if hosts[0] == host:
h[11467]: IndexError: list index out of range

https://pulpito.ceph.com/swagner-2021-07-20_10:26:40-rados:cephadm-wip-swagner2-testing-2021-07-19-1105-distro-basic-smithi/6280743

@sebastian-philipp
Copy link
Contributor

even though the traceback seems unrelated to this particular PR (see https://tracker.ceph.com/issues/51713#note-1 ). we need to fix the Traceback nevertheless in order to get this merged

@liewegas
Copy link
Member

liewegas commented Jul 20, 2021 via email

@tchaikov
Copy link
Contributor

i assume that this change should be merged along with #42406 . so leave it unmerged.

@tchaikov tchaikov merged commit 146c796 into ceph:master Jul 28, 2021
@asannes asannes deleted the ingress-fix branch July 31, 2021 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants