Bug 2182898

Summary: --dns-add command is not functioning [rhel-9.2.0.z]
Product: Red Hat Enterprise Linux 9 Reporter: RHEL Program Management Team <pgm-rhel-tools>
Component: podmanAssignee: Jindrich Novy <jnovy>
Status: CLOSED ERRATA QA Contact: Yuhui Jiang <yujiang>
Severity: high Docs Contact:
Priority: unspecified    
Version: 9.3CC: ajia, arajan, atomic-bugs, bbaude, dwalsh, jligon, jnovy, lsm5, mboddu, mheon, pthomas, tsweeney, umohnani, ypu, yujiang
Target Milestone: rcKeywords: Triaged, ZStream
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: podman-4.4.1-8.el9_2 netavark-1.5.0-3.el9_2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 2182896
: 2188566 (view as bug list) Environment:
Last Closed: 2023-05-09 11:02:52 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2182896    
Bug Blocks: 2188566    

Comment 1 Tom Sweeney 2023-03-29 22:16:38 UTC
We will need backports for common, then create a new version of common, and then backport to Podman v4.4.1-rhel branch and also vendor the updated common in.

Comment 3 Tom Sweeney 2023-04-18 01:26:04 UTC
Fixed with https://github.com/containers/common/pull/1387, which is part of c/common v0.51.2 and Podman v4.4.1-rhel.  Jindrich, please take the top of that Podman branch for this.

Setting to Post and assigning to Jindrich for any further packaging or BZ needs.

Comment 4 Jindrich Novy 2023-04-18 11:04:41 UTC
Tom, this is a c/common PR, do you happen to have a PR against podman?

Comment 5 Tom Sweeney 2023-04-18 14:09:21 UTC
Ooops, actually it went in for two other BZs too.  https://github.com/containers/podman/pull/18006

The BZ's for that were: https://bugzilla.redhat.com/show_bug.cgi?id=2183601 and https://bugzilla.redhat.com/show_bug.cgi?id=2183602  but the patch also addresses this.

Comment 6 Jindrich Novy 2023-04-18 15:30:05 UTC
Thanks Tom! This is actually good news. The functional part is already applied. Only this part of https://github.com/containers/common/pull/1387 is missing:

@@ -666,6 +666,8 @@ var _ = Describe("Config", func() {
 			testNetwork, err := libpodNet.NetworkInspect("test-network")
 			Expect(err).To(BeNil())
 			Expect(testNetwork.NetworkDNSServers).To(Equal([]string{"8.8.8.8", "3.3.3.3", "7.7.7.7"}))
+			err = libpodNet.NetworkUpdate("test-network", types.NetworkUpdateOptions{AddDNSServers: []string{"fake"}})
+			Expect(err).To(HaveOccurred())
 		})
 
 		It("update NetworkDNSServers RemoveDNSServers", func() {
@@ -690,6 +692,8 @@ var _ = Describe("Config", func() {
 			testNetwork, err := libpodNet.NetworkInspect("test-network")
 			Expect(err).To(BeNil())
 			Expect(testNetwork.NetworkDNSServers).To(Equal([]string{"8.8.8.8"}))
+			err = libpodNet.NetworkUpdate("test-network", types.NetworkUpdateOptions{RemoveDNSServers: []string{"fake"}})
+			Expect(err).To(HaveOccurred())
 		})
 
 		It("update NetworkDNSServers Add and Remove DNSServers", func() {

Which led me to a conclusion it's not applied. I will now go ahead do the bugzilla paperwork :)

Comment 7 Jindrich Novy 2023-04-18 15:35:26 UTC
The hunk noted in comment #6 is for libnetwork/netavark/config_test.go - assuming we don't need that one?

Comment 22 errata-xmlrpc 2023-05-09 11:02:52 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (podman bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2023:2620