Skip to content

qa/tests: added quincy-x suite initial draft#45382

Closed
yuriw wants to merge 1 commit intomainfrom
wip-yuriw-quincy-x-master
Closed

qa/tests: added quincy-x suite initial draft#45382
yuriw wants to merge 1 commit intomainfrom
wip-yuriw-quincy-x-master

Conversation

@yuriw
Copy link
Contributor

@yuriw yuriw commented Mar 14, 2022 •
edited
Loading

Signed-off-by: Yuri Weinstein yweinste@redhat.com

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

@yuriw yuriw requested review from adk3798, jdurgin and neha-ojha March 14, 2022 16:18
@yuriw
Copy link
Contributor Author

yuriw commented Mar 14, 2022

@adk3798 @neha-ojha pls suggest regarding use of ceph-adm in this suite

@yuriw
Copy link
Contributor Author

yuriw commented Mar 14, 2022 •
edited
Loading

CEPH_QA_EMAIL="ceph-qa@ceph.io"; CEPH_BRANCH=master; MACHINE_NAME=smithi; teuthology-suite -v -c $CEPH_BRANCH -n 100 -m $MACHINE_NAME -s upgrade/quincy-x/parallel -k distro -e $CEPH_QA_EMAIL --suite-branch wip-yuriw-quincy-x-master -p 70 --force-priority -l 1

http://pulpito.ceph.com/teuthology-2022-03-14_16:32:03-upgrade:quincy-x:parallel-master-distro-default-smithi/

CEPH_QA_EMAIL="ceph-qa@ceph.io"; CEPH_BRANCH=master; MACHINE_NAME=smithi; teuthology-suite -v -c $CEPH_BRANCH -n 100 -m $MACHINE_NAME -s upgrade/quincy-x/stress-split -k distro -e $CEPH_QA_EMAIL --suite-branch wip-yuriw-quincy-x-master -p 70 --force-priority -l 1

http://pulpito.ceph.com/teuthology-2022-03-14_16:33:26-upgrade:quincy-x:stress-split-master-distro-default-smithi/

jdurgin
jdurgin approved these changes Mar 16, 2022
View reviewed changes
Copy link
Member

@jdurgin jdurgin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, just a couple things we can drop with quincy

@yuriw yuriw force-pushed the wip-yuriw-quincy-x-master branch 4 times, most recently from 7ef4f21 to e0d8d97 Compare March 16, 2022 14:58
@yuriw yuriw requested a review from jdurgin March 16, 2022 15:00
@yuriw
Copy link
Contributor Author

yuriw commented Mar 16, 2022

http://pulpito.front.sepia.ceph.com/teuthology-2022-03-16_17:34:28-upgrade:quincy-x:parallel-master-distro-default-smithi

http://pulpito.front.sepia.ceph.com/teuthology-2022-03-16_17:35:05-upgrade:quincy-x:stress-split-master-distro-default-smithi

neha-ojha
neha-ojha reviewed Mar 16, 2022
View reviewed changes
neha-ojha
neha-ojha reviewed Mar 16, 2022
View reviewed changes
neha-ojha
neha-ojha reviewed Mar 16, 2022
View reviewed changes
@yuriw yuriw force-pushed the wip-yuriw-quincy-x-master branch from e0d8d97 to acf7427 Compare March 16, 2022 20:57
neha-ojha
neha-ojha reviewed Mar 16, 2022
View reviewed changes
Copy link
Member

@neha-ojha neha-ojha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The octopus-x tests included:

  • stress-split-no-cephadm
  • stress-split-erasure-code-no-cephadm
  • rgw-multisite
  • parallel
  • stress-split

The quincy-x tests will now have:

  • parallel
  • stress-split

There is a lot of extra test coverage (in terms of different kinds of configuration and thrashing) in octopus-x/stress-split-erasure-code-no-cephadm and stress-split-no-cephadm, which should be ported to the cephadm based upgrade tests. I noticed that the pacific-x upgrade tests are also missing those.

@yuriw yuriw force-pushed the wip-yuriw-quincy-x-master branch from acf7427 to 9240e98 Compare March 17, 2022 16:37
@yuriw
Copy link
Contributor Author

yuriw commented Mar 17, 2022

https://pulpito.ceph.com/teuthology-2022-03-17_16:40:19-upgrade:quincy-x:parallel-master-distro-default-smithi

https://pulpito.ceph.com/teuthology-2022-03-17_16:41:06-upgrade:quincy-x:stress-split-master-distro-default-smithi

@djgalloway
Copy link
Contributor

djgalloway commented Mar 17, 2022

This is wrong:

  tasks:
  - cephadm:
      cephadm_branch: quincy
      cephadm_git_url: https://github.com/ceph/ceph
      conf:
        osd:
          osd_class_default_list: '*'
          osd_class_load_list: '*'
      image: docker.io/ceph/daemon-base:latest-quincy

We haven't pushed dev container images there for months.

@yuriw yuriw force-pushed the wip-yuriw-quincy-x-master branch 3 times, most recently from 4a271f1 to 7afbe34 Compare March 17, 2022 19:52
@yuriw yuriw added the tests label Mar 17, 2022
@ljflores ljflores removed request for a team, jdurgin, nSedrickm and nizamial09 August 11, 2022 17:49
@ljflores
Copy link
Member

ljflores commented Aug 11, 2022

Sorry, attempted a rebase and it went wrong.

@yuriw @ljflores
qa/tests: added quincy-x suite initial draft
91ecc02 
Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
@ljflores
Copy link
Member

ljflores commented Aug 11, 2022

Alright, here is a successful rebase.

@ljflores
Copy link
Member

ljflores commented Aug 11, 2022

jenkins test make check

@neha-ojha neha-ojha mentioned this pull request Aug 22, 2022
Merged
14 tasks
@github-actions
Copy link

github-actions bot commented Sep 15, 2022

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

@yuriw
Copy link
Contributor Author

yuriw commented Oct 4, 2022

reef run:
CEPH_QA_EMAIL="ceph-qa@ceph.io"; CEPH_BRANCH=wip-all-kickoff-r; MACHINE_NAME=smithi; teuthology-suite -v -c $CEPH_BRANCH -n 100 -m $MACHINE_NAME -s upgrade/quincy-x/parallel -k distro -e $CEPH_QA_EMAIL --suite-branch wip-yuriw-quincy-x-master -p 70 --force-priority -l 1 --suite-repo https://github.com/ceph/ceph.git

http://pulpito.front.sepia.ceph.com/teuthology-2022-10-04_17:17:42-upgrade:quincy-x:parallel-wip-all-kickoff-r-distro-default-smithi

@ljflores
Copy link
Member

ljflores commented Oct 4, 2022

@yuriw it's green!

@ljflores
Copy link
Member

ljflores commented Oct 4, 2022

The branch needs to be rebased again though.

@ljflores
Copy link
Member

ljflores commented Oct 4, 2022

Oh, I see-- lots of conflicts after the Reef kickoff PR was merged. I'll need to check how much overlap there is. This PR can be closed if the Reef kickoff covers everything.

@github-actions
Copy link

github-actions bot commented Oct 5, 2022

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

@yuriw
Copy link
Contributor Author

yuriw commented Oct 5, 2022 •
edited
Loading

run on main

http://pulpito.front.sepia.ceph.com:80/teuthology-2022-10-05_14:50:39-upgrade:quincy-x-main-distro-default-smithi/

(as: teuthology@teuthology:~$ teuthology-suite -v --ceph-repo $CEPH_REPO --suite-repo $CEPH_REPO -c $CEPH_BRANCH -m $MACHINE_NAME -s upgrade/quincy-x -k $DISTRO -p $PRIO -e $CEPH_QA_MAIL --suite-branch $CEPH_BRANCH -l $LIMIT --force-priority --subset 111/120000 -n 17)

I think we can close this PR

@ljflores
Copy link
Member

ljflores commented Oct 5, 2022

All green from the Reef kickoff PR:
http://pulpito.front.sepia.ceph.com/lflores-2022-10-04_22:22:19-upgrade:quincy-x-wip-all-kickoff-r-distro-default-smithi/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@neha-ojha neha-ojha neha-ojha left review comments

@adk3798 adk3798 adk3798 left review comments

@jdurgin jdurgin jdurgin approved these changes

@ljflores ljflores ljflores approved these changes

Assignees

No one assigned

Labels

needs-rebase tests

Projects

Ceph-Dashboard
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@yuriw @djgalloway @ljflores @jdurgin @neha-ojha @adk3798