Bug 2232482

Summary: odf must-gather does not collect multus NetworkAttachmentDefinitions
Product: [Red Hat Storage] Red Hat OpenShift Data Foundation Reporter: yati padia <ypadia>
Component: must-gatherAssignee: yati padia <ypadia>
Status: NEW --- QA Contact: Prasad Desala <tdesala>
Severity: low Docs Contact:
Priority: unspecified    
Version: 4.13CC: brgardne, etamir, muagarwa, odf-bz-bot, oviner, ypadia
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 2218309 Environment:
Last Closed: 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: 2218309    
Bug Blocks:    

Description yati padia 2023-08-17 05:15:17 UTC
+++ This bug was initially created as a clone of Bug #2218309 +++

Description of problem (please be detailed as possible and provide log
snippests):

The ODF must-gather does not collect NetworkAttachmentDefinitions that affect Multus. The must-gather should be updated to collect NetworkAttachmentDefinition resources from the "default" and "openshift-storage" namespaces.


Version of all relevant components (if applicable):

I would hope this can be part of an upcoming 4.13.z release.


Does this issue impact your ability to continue to work with the product
(please explain in detail what is the user impact)?

No.


Is there any workaround available to the best of your knowledge?

OCP must-gather collects NetworkAttachmentDefinitions


Rate from 1 - 5 the complexity of the scenario you performed that caused this
bug (1 - very simple, 5 - very complex)?
1


Can this issue reproducible? Yes.


Can this issue reproduce from the UI? N/A


If this is a regression, please provide more details to justify this: N/A


Steps to Reproduce:
It is not necessary to install ODF with multus enabled to repro!

First, create these two NetworkAttachmentDefinitions:

  apiVersion: k8s.cni.cncf.io/v1
  kind: NetworkAttachmentDefinition
  metadata:
    name: cluster-net
    namespace: openshift-storage
  spec:
    config: '{ "cniVersion": "0.3.1", "type": "macvlan", "master": "br-ex", "mode":
      "bridge", "ipam": { "type": "whereabouts", "range": "192.168.30.0/24" } }'

  apiVersion: k8s.cni.cncf.io/v1
  kind: NetworkAttachmentDefinition
  metadata:
    name: cluster-net
  spec:
    config: '{ "cniVersion": "0.3.1", "type": "macvlan", "master": "br-ex", "mode":
      "bridge", "ipam": { "type": "whereabouts", "range": "192.168.30.0/24" } }'


Next, collect an ODF must-gather

Both NADs should be collected but are not. The first NAD is in the openshift-storage NS, and the second is in the default NS. NADs from other NSes do not need to be collected for ODF.

--- Additional comment from RHEL Program Management on 2023-06-28 17:41:19 UTC ---

This bug having no release flag set previously, is now set with release flag 'odf‑4.14.0' to '?', and so is being proposed to be fixed at the ODF 4.14.0 release. Note that the 3 Acks (pm_ack, devel_ack, qa_ack), if any previously set while release flag was missing, have now been reset since the Acks are to be set against a release flag.

--- Additional comment from Blaine Gardner on 2023-07-19 17:15:08 UTC ---

I see this is merged into the m-g main codebase and is in 4.14. 

@etamir would you like to target a 4.13.z backport as well? 
What about 4.12.z for any support exceptions?

--- Additional comment from Red Hat Bugzilla on 2023-08-03 08:28:10 UTC ---

Account disabled by LDAP Audit

--- Additional comment from Eran Tamir on 2023-08-15 16:22:03 UTC ---

Yes. It makes sense to backport to 4.13 and also 4.12 (lower priority)

--- Additional comment from Eran Tamir on 2023-08-15 16:22:23 UTC ---

Yes. It makes sense to backport to 4.13 and also 4.12 (lower priority)

--- Additional comment from Blaine Gardner on 2023-08-15 17:42:03 UTC ---

@ypadia would you be able to take the action to make sure backport BZs get created for 4.13.z and 4.12.z? I'd like to get this off of my todo list to keep focusing on Rook BZs. Thanks :)

--- Additional comment from Mudit Agarwal on 2023-08-16 04:35:55 UTC ---

Yati, please create clones for 4.13/4.12. Reach out to Sunil for adding the backports to the respective z-streams.

Oded, please provide qa_ack

--- Additional comment from yati padia on 2023-08-16 05:50:46 UTC ---

Added the backport PR for both 4.13 and 4.12

--- Additional comment from Mudit Agarwal on 2023-08-17 05:07:45 UTC ---

Yati, please create clones bugs for 4.13/4.12