Skip to content
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

common: add centos8 support #4798

Merged
merged 9 commits into from
Jan 8, 2020
Merged

common: add centos8 support #4798

merged 9 commits into from
Jan 8, 2020

Conversation

guits
Copy link
Collaborator

@guits guits commented Dec 2, 2019

Ceph octopus only supports CentOS 8.

This commit adds CentOS 8 support:

  • update vagrant image in tox configurations.
  • add CentOS 8 repository for el8 dependencies.
  • CentOS 8 container engine is podman (same than RHEL 8).
  • don't use the epel mirror on sepia because it's epel7 only.

Signed-off-by: Guillaume Abrioux [email protected]
Co-authored-by: Dimitri Savineau [email protected]

@guits guits added the DNM Do NOT merge label Dec 2, 2019
@guits
Copy link
Collaborator Author

guits commented Dec 2, 2019

wip

@dsavineau
Copy link
Contributor

@chkumar246
Copy link

check-rdo

@softwarefactory-project-zuul
Copy link

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.

@dsavineau dsavineau changed the title [skip ci] common: add centos8 support common: add centos8 support Dec 19, 2019
@dsavineau dsavineau force-pushed the guits-centos8 branch 4 times, most recently from d7cf29f to 3667af0 Compare December 19, 2019 16:15
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-lvm_batch

@dsavineau dsavineau force-pushed the guits-centos8 branch 3 times, most recently from 9847f05 to ca55093 Compare December 19, 2019 21:07
@dsavineau dsavineau force-pushed the guits-centos8 branch 2 times, most recently from 56f5e39 to 6502fa3 Compare December 23, 2019 15:47
@dsavineau dsavineau force-pushed the guits-centos8 branch 2 times, most recently from a4eb4a4 to fb0d08c Compare January 2, 2020 20:57
@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-container-update

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-container-lvm_batch

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-non_container-update

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-container-update

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-container-ooo_collocation

@guits
Copy link
Collaborator Author

guits commented Jan 3, 2020

jenkins test centos-non_container-all_daemons

1 similar comment
@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-container-lvm_batch

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-switch_to_containers

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-lvm_osds

1 similar comment
@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-lvm_osds

guits and others added 4 commits January 6, 2020 10:02
Ceph octopus only supports CentOS 8.

This commit adds CentOS 8 support:
  - update vagrant image in tox configurations.
  - add CentOS 8 repository for el8 dependencies.
  - CentOS 8 container engine is podman (same than RHEL 8).
  - don't use the epel mirror on sepia because it's epel7 only.

Signed-off-by: Guillaume Abrioux <[email protected]>
Co-authored-by: Dimitri Savineau <[email protected]>
Since we don't have nfs-ganesha builds available on CentOS 8 at the
moment on shaman then we can use the alternative repository at [1]

[1] https://download.nfs-ganesha.org/3/LATEST/CentOS

Signed-off-by: Dimitri Savineau <[email protected]>
Before this patch, the lvm2 package installation was done during the
ceph-osd role.
However we were running ceph-volume command in the ceph-config role
before ceph-osd. If lvm2 wasn't installed then the ceph-volume command
fails:

error checking path "/run/lock/lvm": stat /run/lock/lvm: no such file or
directory

This wasn't visible before because lvm2 was automatically installed as
docker dependency but it's not the same for podman on CentOS 8.

Signed-off-by: Dimitri Savineau <[email protected]>
Since RHEL 8.1 we need to add the ganesha_t type to the permissive
SELinux list.
Otherwise the nfs-ganesha service won't start.
This was done on RHEL 7 previously and part of the nfs-ganesha-selinux
package on RHEL 8.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1786110

Signed-off-by: Dimitri Savineau <[email protected]>
@dsavineau dsavineau self-requested a review January 6, 2020 15:08
@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-container-update

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-update

@guits
Copy link
Collaborator Author

guits commented Jan 6, 2020

jenkins test centos-non_container-lvm_batch

@guits guits removed the DNM Do NOT merge label Jan 7, 2020
@guits
Copy link
Collaborator Author

guits commented Jan 7, 2020

jenkins test centos-container-purge

1 similar comment
@dsavineau
Copy link
Contributor

jenkins test centos-container-purge

dsavineau and others added 4 commits January 7, 2020 11:08
Some docker commands were hardcoded in tests playbooks and some
conditions were not taking care of the containerized_deployment
variable but only the atomic fact.

Signed-off-by: Dimitri Savineau <[email protected]>
on master we can't test upgrade from stable-4.0/CentOS 7 to
master/CentOS 8.

This commit refact the upgrade so we test upgrade from master/CentOS 8
to master/CentOS 8 (octopus to octopus)

Signed-off-by: Guillaume Abrioux <[email protected]>
nfs-ganesha makes the CI failing because of issue related to SELinux.

See:
- https://bugzilla.redhat.com/show_bug.cgi?id=1788563
- nfs-ganesha/nfs-ganesha#527

Until we can get this fixed, let's disable nfs-ganesha testing
temporarily.

Signed-off-by: Guillaume Abrioux <[email protected]>
The rtslib python library is now available in the distribution so we
shouldn't have to use the shaman repository

Signed-off-by: Dimitri Savineau <[email protected]>
Remove all stopped/exited containers.

Signed-off-by: Dimitri Savineau <[email protected]>
@dsavineau
Copy link
Contributor

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jan 8, 2020

tears came to my eyes this morning when I saw the CI passing

@guits guits merged commit 42366f0 into master Jan 8, 2020
@guits guits deleted the guits-centos8 branch January 8, 2020 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants