Skip to content

quincy: mgr/dashboard: unselect rows in datatables #46323

Merged
epuertat merged 1 commit intoceph:quincyfrom
rhcs-dashboard:wip-55691-quincy
May 31, 2022
Merged

quincy: mgr/dashboard: unselect rows in datatables #46323
epuertat merged 1 commit intoceph:quincyfrom
rhcs-dashboard:wip-55691-quincy

Conversation

@Sarthak0702
Copy link
Member

backport tracker: https://tracker.ceph.com/issues/55691


backport of #45313
parent tracker: https://tracker.ceph.com/issues/53244

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

Fixes: https://tracker.ceph.com/issues/53244
Signed-off-by: Sarthak0702 <sarthak.0702@gmail.com>
(cherry picked from commit b79e2a6)
@Sarthak0702 Sarthak0702 requested a review from a team as a code owner May 18, 2022 18:17
@Sarthak0702 Sarthak0702 requested review from avanthakkar and epuertat and removed request for a team May 18, 2022 18:17
@Sarthak0702 Sarthak0702 added this to the quincy milestone May 18, 2022
@Sarthak0702 Sarthak0702 added the backport: no-conflicts Backport without conflicts label May 18, 2022
@Sarthak0702 Sarthak0702 requested a review from nizamial09 May 19, 2022 12:29
@epuertat
Copy link
Member

@Sarthak0702 you need to trigger dashboard and cephadm tests here. It's not enough with the "jenkins test ..." thing, you first have to rename your PR (just add a trailing whitespace). I'll do that for you this time ;-)

@epuertat
Copy link
Member

epuertat commented May 19, 2022

jenkins test dashboard

@epuertat
Copy link
Member

Confirmed: it doesn't work. Now let's rename the PR.

@epuertat epuertat changed the title quincy: mgr/dashboard: unselect rows in datatables quincy: mgr/dashboard: unselect rows in datatables May 19, 2022
@epuertat
Copy link
Member

jenkins test dashboard

@epuertat
Copy link
Member

There they go

@epuertat
Copy link
Member

jenkins test dashboard cephadm

@Sarthak0702
Copy link
Member Author

@epuertat Thanks for this.!

@Sarthak0702
Copy link
Member Author

jenkins test dashboard cephadm

3 similar comments
@Sarthak0702
Copy link
Member Author

jenkins test dashboard cephadm

@Sarthak0702
Copy link
Member Author

jenkins test dashboard cephadm

@epuertat
Copy link
Member

jenkins test dashboard cephadm

@epuertat epuertat merged commit 65a07c8 into ceph:quincy May 31, 2022
@epuertat epuertat deleted the wip-55691-quincy branch May 31, 2022 12:39
nizamial09 added a commit to rhcs-dashboard/ceph that referenced this pull request Sep 4, 2022
…er found it

This looks related to ceph#46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
nizamial09 added a commit to rhcs-dashboard/ceph that referenced this pull request Sep 16, 2022
…er found it

This looks related to ceph#46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit dcbf179)
nizamial09 added a commit to rhcs-dashboard/ceph that referenced this pull request Sep 16, 2022
…er found it

This looks related to ceph#46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit dcbf179)
zalsader pushed a commit to zalsader/ceph that referenced this pull request Sep 19, 2022
…er found it

This looks related to ceph#46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
Pegonzal pushed a commit that referenced this pull request Sep 21, 2022
…er found it

This looks related to #46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
Pegonzal pushed a commit that referenced this pull request Oct 13, 2022
…er found it

This looks related to #46323, but i am
not sure why it started failing recently in the teuthology.

The error is fixed by deselecting the row manually, so that it won't
accidentally get deselected when we actually want it to select it.

Fixes: https://tracker.ceph.com/issues/57207
Signed-off-by: Nizamudeen A <nia@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants