Bug 1855224 - [Bare Metal IPI] After upgrade from OCP4.4 to 4.5 SRV record wasn't cleaned up
Summary: [Bare Metal IPI] After upgrade from OCP4.4 to 4.5 SRV record wasn't cleaned up
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Networking
Version: 4.5
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: 4.6.0
Assignee: Ben Nemec
QA Contact: Victor Voronkov
Depends On:
Blocks: 1855373
TreeView+ depends on / blocked
Reported: 2020-07-09 09:49 UTC by Victor Voronkov
Modified: 2020-09-25 21:20 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1855373 (view as bug list)
Last Closed: 2020-09-25 21:20:55 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Github openshift machine-config-operator pull 1917 0 None closed Bug 1855373: etcd: remove etcd entries from mdns 2021-02-18 12:05:00 UTC

Description Victor Voronkov 2020-07-09 09:49:47 UTC
Description of problem:
After upgrade from OCP4.4 to 4.5 SRV record wasn't cleaned up and potentially may conflict with masters data available via KUBE API

Version-Release number of selected component (if applicable):

How reproducible:
Upgrade from OCP4.4 to 4.5

Steps to Reproduce:
1. Deploy OCP 4.4 which should include SRV record
2. Upgrade to 4.5
3. host -t SRV _etcd-server-ssl._tcp.ocp-edge-cluster-0.qe.lab.redhat.com

Actual results:
[core@master-0-0 ~]$ host -t SRV _etcd-server-ssl._tcp.ocp-edge-cluster-0.qe.lab.redhat.com
_etcd-server-ssl._tcp.ocp-edge-cluster-0.qe.lab.redhat.com has SRV record 0 10 2380 master-0-1.ocp-edge-cluster-0.qe.lab.redhat.com.
_etcd-server-ssl._tcp.ocp-edge-cluster-0.qe.lab.redhat.com has SRV record 0 10 2380 master-0-0.ocp-edge-cluster-0.qe.lab.redhat.com.
_etcd-server-ssl._tcp.ocp-edge-cluster-0.qe.lab.redhat.com has SRV record 0 10 2380 master-0-2.ocp-edge-cluster-0.qe.lab.redhat.com.

Expected results:
No SRV record should exist in OCP 4.5 deployment

Additional info:

[kni@provisionhost-0-0 ~]$ oc version
Client Version: 4.4.10
Server Version: 4.4.0-0.nightly-2020-07-04-051327
Kubernetes Version: v1.17.1+a1af596

Comment 1 Ben Bennett 2020-07-09 13:45:12 UTC
Setting the target to the current development branch.  We can work out whether to do a backport once the issue is identified and resolved in the main branch.

Comment 2 Ben Nemec 2020-07-09 14:05:18 UTC
This was never removed on 4.5: https://github.com/openshift/machine-config-operator/blob/4173030d89fbf4a7a0976d1665491a4d9a6e54f1/templates/master/00-master/baremetal/files/baremetal-mdns-config.yaml#L9

It looks like the dependency in baremetal-runtimecfg made it in for 4.5 though, so all we should have to do is backport https://github.com/openshift/machine-config-operator/pull/1556.

Comment 4 Ben Nemec 2020-07-09 17:17:35 UTC
For 4.6 this was fixed by https://github.com/openshift/machine-config-operator/pull/1556. Will work on the 4.5 backport in the clone.

Note You need to log in before you can comment on or make changes to this bug.