Skip to content

octopus: qa/suites/rados/cephadm/upgrade: change starting version by distro#40364

Merged
tchaikov merged 1 commit intoceph:octopusfrom
ideepika:wip-bug-48142-octopus
Apr 27, 2021
Merged

octopus: qa/suites/rados/cephadm/upgrade: change starting version by distro#40364
tchaikov merged 1 commit intoceph:octopusfrom
ideepika:wip-bug-48142-octopus

Conversation

@ideepika
Copy link
Member

@ideepika ideepika commented Mar 24, 2021

centos/rhel have podman 2, which does not like conflicting --cap-add and
--privileged arguments. cephadm versions prior to 15.2.5 use both args,
however, which means the rhel/centos upgrade test has to start at 15.2.5
to work at all on those distros (with the updated podman).

Fixes: https://tracker.ceph.com/issues/48142
Signed-off-by: Sage Weil sage@newdream.net
(cherry picked from commit 38e14f9)

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@liewegas
qa/suites/rados/cephadm/upgrade: change starting version by distro
4ea5687 
centos/rhel have podman 2, which does not like conflicting --cap-add and
--privileged arguments.  cephadm versions prior to 15.2.5 use both args,
however, which means the rhel/centos upgrade test has to start at 15.2.5
to work at all on those distros (with the updated podman).

Fixes: https://tracker.ceph.com/issues/48142
Signed-off-by: Sage Weil <sage@newdream.net>
(cherry picked from commit 38e14f9)
@ideepika ideepika requested a review from a team as a code owner March 24, 2021 13:07
@github-actions github-actions bot added this to the octopus milestone Mar 24, 2021
@ideepika
Copy link
Member Author

ideepika commented Mar 31, 2021

@yuriw can you include this pr in your next octopus batch?

@ideepika
Copy link
Member Author

ideepika commented Apr 5, 2021

hey @Sage!
the issue still persists on Octopus in testing branch with patch which is meant to resolve: https://tracker.ceph.com/issues/48142

2021-04-01T15:48:43.375 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 podman[32767]: Error: no container with name or ID ceph-716172b6-9301-11eb-8175-001a4aab830c-mon.c found: no such container
2021-04-01T15:48:43.375 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 systemd[1]: Started Ceph mon.c for 716172b6-9301-11eb-8175-001a4aab830c.
2021-04-01T15:48:43.375 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 bash[32820]: Error: failed to evict container: "": failed to find container "ceph-716172b6-9301-11eb-8175-001a4aab830c-mon.c" in state: no container with name or ID ceph-716172b6-9301-11eb-8175-001a4aab830c-mon.c found: no such container
2021-04-01T15:48:43.376 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 bash[32820]: Error: failed to evict container: "": failed to find container "ceph-716172b6-9301-11eb-8175-001a4aab830c-mon.c" in state: no container with name or ID ceph-716172b6-9301-11eb-8175-001a4aab830c-mon.c found: no such container
2021-04-01T15:48:43.376 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 bash[32820]: Error: invalid config provided: CapAdd and privileged are mutually exclusive options
2021-04-01T15:48:43.377 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 systemd[1]: ceph-716172b6-9301-11eb-8175-001a4aab830c@mon.c.service: Main process exited, code=exited, status=125/n/a
2021-04-01T15:48:43.377 INFO:journalctl@ceph.mon.c.smithi073.stdout:Apr 01 15:48:43 smithi073 systemd[1]: ceph-716172b6-9301-11eb-8175-001a4aab830c@mon.c.service: Failed with result 'exit-code'.
2021-04-01T15:48:44.837 INFO:teuthology.orchestra.run.smithi073.stdout:

/ceph/teuthology-archive/yuriw-2021-04-01_15:23:17-rados-wip-yuri-testing-2021-03-31-1516-octopus-distro-basic-smithi/6014937/teuthology.log
can you take a look what we might be missing?

@tchaikov
Copy link
Contributor

tchaikov commented Apr 14, 2021

i think most of cephadm test failures are caused by https://tracker.ceph.com/issues/49293

@ideepika
Copy link
Member Author

ideepika commented Apr 14, 2021

i think most of cephadm test failures are caused by https://tracker.ceph.com/issues/49293

i think most of cephadm test failures are caused by https://tracker.ceph.com/issues/49293

this change looks good overall, I was only concerned whether the patch for https://tracker.ceph.com/issues/48142 is working in #40364 (comment) but we can merge this and see later if your tests looks good!

@idryomov idryomov changed the base branch from octopus to octopus-saved April 20, 2021 18:12
@idryomov idryomov changed the base branch from octopus-saved to octopus April 20, 2021 18:12
@tchaikov
Copy link
Contributor

tchaikov commented Apr 25, 2021

jenkins test api

@tchaikov
Copy link
Contributor

tchaikov commented Apr 25, 2021

jenkins test make check

@tchaikov tchaikov merged commit e218b70 into ceph:octopus Apr 27, 2021
@ideepika ideepika deleted the wip-bug-48142-octopus branch May 5, 2021 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@liewegas liewegas liewegas approved these changes

@sebastian-philipp sebastian-philipp Awaiting requested review from sebastian-philipp

Assignees

No one assigned

Labels

cephadm octopus-batch-1

Projects

None yet

Milestone

octopus

Development

Successfully merging this pull request may close these issues.

4 participants

@ideepika @tchaikov @liewegas @yuriw