Skip to content

pacific: qa: run e2e test on centos only#49337

Merged
ljflores merged 1 commit intoceph:pacificfrom
ljflores:wip-tracker-54992
Jan 4, 2023
Merged

pacific: qa: run e2e test on centos only#49337
ljflores merged 1 commit intoceph:pacificfrom
ljflores:wip-tracker-54992

Conversation

@ljflores
Copy link
Member

@ljflores ljflores commented Dec 8, 2022

Backport of #42158

Fixes: https://tracker.ceph.com/issues/54992

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
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 dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

this change is a follow up of 02b8b0f,
which failed to remove the random facet for distro.

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 15fa32d)
@ljflores ljflores requested a review from tchaikov December 8, 2022 21:18
@ljflores ljflores requested a review from a team as a code owner December 8, 2022 21:18
@github-actions github-actions bot added this to the pacific milestone Dec 8, 2022
@ljflores
Copy link
Member Author

ljflores commented Dec 8, 2022

@tchaikov pacific dashboard tests have been failing when run on ubuntu, and I noticed that we removed support for ubuntu in releases <= quincy.

This is the commit that seems to take care of it, but do you know if this is sufficient, or if something more/different needs to be backported?

@ljflores
Copy link
Member Author

ljflores commented Dec 8, 2022

@ljflores ljflores requested review from a team, aaSharma14, avanthakkar, pereman2 and torchiaf and removed request for a team December 9, 2022 15:52
@ljflores
Copy link
Member Author

jenkins test make check

@tchaikov
Copy link
Contributor

@tchaikov pacific dashboard tests have been failing when run on ubuntu, and I noticed that we removed support for ubuntu in releases >= quincy.

This is the commit that seems to take care of it, but do you know if this is sufficient, or if something more/different needs to be backported?

@ljflores hi Laura, sorry i don't have enough bandwidth looking into this issue so i cannot provide an answer to your inquiries.

@tchaikov tchaikov removed their request for review December 21, 2022 13:16
@ljflores
Copy link
Member Author

ljflores commented Jan 3, 2023

@tchaikov no worries!

@ceph/dashboard can you please take a look?

@nizamial09
Copy link
Member

pacific dashboard tests have been failing when run on ubuntu, and I noticed that we removed support for ubuntu in releases >= quincy.

@ljflores do you mean to say releases <=quincy? if that's the case i am okay with it..

@ljflores
Copy link
Member Author

ljflores commented Jan 4, 2023

@nizamial09 yes, apologies, that should say <= quincy. Thanks for reviewing!

@ljflores
Copy link
Member Author

ljflores commented Jan 4, 2023

As this has already been tested on teuthology (http://pulpito.front.sepia.ceph.com/lflores-2022-12-08_21:33:17-rados:dashboard-pacific-distro-default-smithi/) I will merge this.

@ljflores ljflores merged commit 708b6e5 into ceph:pacific Jan 4, 2023
@ljflores ljflores deleted the wip-tracker-54992 branch January 4, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants