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

mgr/dashboard: Fix failing config dashboard e2e check #43210

Merged
merged 1 commit into from Sep 21, 2021

Conversation

nizamial09
Copy link
Member

@nizamial09 nizamial09 commented Sep 17, 2021

Recently a new osd config has been added in
6ca32bd and this is getting Modified.
So on our dashboard e2e config check which checks for the Modified
filter, this is also coming as one of the entry. So we need to increase the
count.

Fixes: https://tracker.ceph.com/issues/52649
Signed-off-by: Nizamudeen A nia@redhat.com

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 dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@nizamial09 nizamial09 requested a review from a team as a code owner September 17, 2021 16:10
@nizamial09 nizamial09 added this to In progress in Dashboard via automation Sep 17, 2021
Dashboard automation moved this from In progress to Reviewer approved Sep 17, 2021
@nizamial09
Copy link
Member Author

jenkins test dashboard

@nizamial09 nizamial09 added the skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology label Sep 17, 2021
@nizamial09
Copy link
Member Author

jenkins test dashboard

1 similar comment
@nizamial09
Copy link
Member Author

jenkins test dashboard

Recently a new osd config has been added in
6ca32bd and this is getting Modified.
So on our dashboard e2e config check which checks for the Modified
filter, this is also coming on the entry. So we need to increase the
count.

Fixes: https://tracker.ceph.com/issues/52649
Signed-off-by: Nizamudeen A <nia@redhat.com>
@nizamial09 nizamial09 moved this from Reviewer approved to Ready-to-merge in Dashboard Sep 21, 2021
@epuertat epuertat merged commit fef5ab7 into ceph:master Sep 21, 2021
Dashboard automation moved this from Ready-to-merge to Done Sep 21, 2021
@epuertat epuertat deleted the config-e2e-failure branch September 21, 2021 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dashboard needs-review pybind skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology
Projects
Archived in project
Dashboard
  
Done
5 participants