-
Notifications
You must be signed in to change notification settings - Fork 86
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
Add NFS network for Manila to uni04delta #376
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fultonj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Manila tests need to connect to the share for both cephfs and ganesha tests, and they use a special (openstack) network for that [1]. This patch adds the storage NFS network (storagenfs) with VLAN 24 and range 172.21.0.0/24 in the reproducers networking-definition.yml. It also adds a multus range for this network so that the Tempest pod can access this network for testing. The storage network is added to the OCP nodes for the same reason. TODO: update the Ceph playbook and roles to use this network. [1] https://opendev.org/openstack/manila-tempest-plugin/src/branch/master/manila_tempest_tests/config.py#L99 Jira: https://issues.redhat.com/browse/OSPRH-7417 Depends-On: openstack-k8s-operators/architecture#376 Signed-off-by: John Fulton <[email protected]>
When I run the kustomize commands documented at: https://github.com/openstack-k8s-operators/architecture/tree/main/examples/va/hci#stages They produce the desired k8s manifests which are the same but have new storagenfs network: https://paste.openstack.org/show/bM6mLzUtCv0OFpjgBV4r Though, I do see VLAN 21 where 24 was desired on line 80 above so I need to fix that next. Also, maybe I need to update the CI tests to work with the change? |
@fultonj in theory this additional network is required in uni-delta (external ceph + manila w/ ganesha). |
OK, thanks. I'll move it to uni-delta |
571d5d7
to
2e34078
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/c2eb70ccb80545fd9ced7c7b6530b813 ✔️ noop SUCCESS in 0s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/71d89153136f49ecab9affb220ef60dc ✔️ noop SUCCESS in 0s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/7a5800f1986b421a813c034e40c77aca ✔️ noop SUCCESS in 0s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/aee7effdd8a74b68a8847e68d53b9aae ✔️ noop SUCCESS in 0s |
8e59010
to
4cb3db3
Compare
Todo: With this change manilaShares:
share1:
networkAttachments:
- storage but |
The following change adds |
dt/uni04delta/kustomization.yaml
Outdated
name: controlplane | ||
patch: |- | ||
- op: add | ||
path: /spec/manila/template/manilaShares/share1/networkAttachments/- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need a NAD on the storageNFS network because the new Manila driver interacts w/ the ceph mgr nfs module. For this reason manila-share only needs to reach the Ceph public network (storage), while the EDPM nodes needs direct access to StorageNFS because the workload needs to mount the share provided by ceph-nfs cluster.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the manila share pod does not need to be able to access the cifmw_cephadm_nfs_vip
, then I can remove this line. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct, we don't need to add this additional nad, so I guess L40-L48 can be removed.
cifmw_cephadm_nfs_vip
is required on the Ceph side (hence we need it on the edpm nodes), and that network should be "abstracted" at tenant level to allow the workload mount the share using cifmw_cephadm_nfs_vip
.
Manila itself just need to connect to ceph, and the driver actually uses the ceph nfs ...
interface (mgr plugin) to do crud operations against ganesha (this removed the old dbus dependency we had in TripleO).
todo:
|
9c9b1ab
to
88030af
Compare
Manila Tempest tests need to connect to the share for Ganesha via a special (openstack) network [1]. This patch adds the NFS storage network with VLAN 24 and range 172.21.0.0/24 in uni04delta. The NFS network is connected to Ceph and Compute EDPM nodes. A NNCP, NAD, L2Advertisement and IPAddressPool are defined for the NFS network so that a pod in k8s can connect to it; such as the tempest pod which will perform the storage tests and the manilaShares pod(s). In order to make these changes, uni04delta now keeps its own copy of the nncp and networking directories since they differ (by the new network) from the generic ones in the lib directory. [1] https://opendev.org/openstack/manila-tempest-plugin/src/branch/master/manila_tempest_tests/config.py#L99 Jira: https://issues.redhat.com/browse/OSPRH-7417 Depends-On: openstack-k8s-operators/ci-framework#2273 Signed-off-by: John Fulton <[email protected]>
Obsoleted by #405 |
Manila Tempest tests need to connect to the share for Ganesha via a special (openstack) network [1].
This patch adds the NFS storage network with VLAN 24 and range 172.21.0.0/24 in uni04delta. The NFS network is connected to Ceph and Compute EDPM nodes. A NNCP, NAD, L2Advertisement and IPAddressPool are defined for the NFS network so that a pod in k8s can connect to it; such as the tempest pod which will perform the storage tests and the manilaShares pod(s).
In order to make these changes, uni04delta now keeps its own copy of the nncp and networking directories since they differ (by the new network) from the generic ones in the lib directory.
[1] https://opendev.org/openstack/manila-tempest-plugin/src/branch/master/manila_tempest_tests/config.py#L99
Jira: https://issues.redhat.com/browse/OSPRH-7417
Depends-On: openstack-k8s-operators/ci-framework#2273