Enable NFS server as cinder backend for master job#217
Conversation
|
Skipping CI for Draft Pull Request. |
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
2 times, most recently
from
682e745 to
b7d92c0
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bd9c1aeda2594bf1815245993de81b20 ✔️ openstack-meta-content-provider-master SUCCESS in 1h 49m 04s |
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
5 times, most recently
from
bdf51c1 to
ea2e41d
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d3f2b4ca962c4c578b6ac9f8c360a33a ✔️ openstack-meta-content-provider-master SUCCESS in 3h 04m 18s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/95c2626580bf4e97a1b8bfe64df99a3d ✔️ openstack-meta-content-provider-master SUCCESS in 3h 05m 13s |
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
from
ea2e41d to
dae6c8e
Compare
|
This change depends on a change that failed to merge. Change #221 is needed. |
|
recheck |
1 similar comment
|
recheck |
|
This change depends on a change that failed to merge. Change https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954625 is needed. |
|
recheck |
1 similar comment
|
recheck |
|
This change depends on a change that failed to merge. Change https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954625 is needed. |
|
recheck |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
from
dae6c8e to
d4d8e71
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/190b710637624608b30efd6dadb19fb8 ✔️ openstack-meta-content-provider-master SUCCESS in 2h 51m 09s |
|
recheck |
1 similar comment
|
recheck |
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
from
6ed0471 to
12fd3bd
Compare
|
This change depends on a change that failed to merge. Change #275 is needed. |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ce47d1cd1de24599b61b71d36e5c472e ✔️ openstack-meta-content-provider-master SUCCESS in 4h 27m 05s |
|
volume migration tests are passing with the nfs configuration: |
|
recheck |
cescgina
changed the title
| nas_secure_file_operations=false | ||
| nas_secure_file_permissions=false | ||
| customServiceConfigSecrets: | ||
| - cinder-volume-nfs-secrets |
There was a problem hiding this comment.
this all looks fine until here
normally if you have 2 nfs poosl they woudl not use the same storage
you have both of them refencing the same secret
which looks liek this
cinder-volume-nfs-secrets: |
[nfs]
nas_host={{ cifmw_nfs_ip }}
nas_share_path={{ '/data/' + cifmw_nfs_shares[0] }}
we woudl ideally have one secree per nfs backend so that they are actully using diffnt strage lcoations on the backing nfs server or on seperate nfs servers.
There was a problem hiding this comment.
thanks, I was meant to do this but I forgot. I've updated the PR adding the additional share and creating separate secrets, one per pool
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
2 times, most recently
from
3ef0058 to
099a3cd
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/631d1bcb0eab4bc296de60d773e7ae09 ✔️ openstack-meta-content-provider-master SUCCESS in 2h 57m 04s |
Deploy an NFS server in a compute node using cifmw and use it as backend for cinder. This is needed to test volume migrations and depending on the result we might enable it in all jobs or only in some.
cescgina
force-pushed
the
enabled_nfs_server_cinder_backend
branch
from
099a3cd to
45d3689
Compare
SeanMooney
left a comment
SeanMooney
left a comment
There was a problem hiding this comment.
ok ya this version looks good to me
thanks for updating it
|
|
||
| - name: Create the definition of the secrets | ||
| vars: | ||
| share: "{{ cifmw_nfs_shares[indx] }}" |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amoralej The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-merge-bot
Bot
merged commit 9c9ab71
into
openstack-k8s-operators:main
6 participants
Deploy an NFS server in a compute node using cifmw and use it as backend
for cinder. This is needed to test volume migrations and depending on
the result we might enable it in all jobs or only in some.
Depends-On: https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/958644
Depends-On: #275