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 rbd snapshot creation #49531

Merged
merged 1 commit into from Apr 14, 2023

Conversation

aaSharma14
Copy link
Contributor

@aaSharma14 aaSharma14 commented Dec 21, 2022

There are two types of snapshots that can be created on a snapshot based mirroring image - Normal Snapshot(same as journal based snapshot) and Mirror Image Snapshot. Till now Dashboard allowed only Mirror image snapshot, this PR intends to enable both the types

Screenshot from 2023-04-10 18-57-28
Screenshot from 2023-04-10 18-57-43

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

Copy link
Member

@nizamial09 nizamial09 left a comment

Choose a reason for hiding this comment

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

some minor suggestions. if you want you can take that as an improvment lateron, but please make sure to have a tracker on that.

Copy link

@sunilangadi2 sunilangadi2 left a comment

Choose a reason for hiding this comment

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

@aaSharma14 can you please point me to that docs link for these types of snapshot creation?

@aaSharma14
Copy link
Contributor Author

@aaSharma14 can you please point me to that docs link for these types of snapshot creation?

@sunilangadi2 , I hope you are aware of the normal snapshot creation with command - rbd snap create.., the second one is mentioned here - https://docs.ceph.com/en/quincy/rbd/rbd-mirroring/#create-image-mirror-snapshots

@aaSharma14
Copy link
Contributor Author

jenkins test dashboard

@sunilangadi2
Copy link

@aaSharma14 can you please point me to that docs link for these types of snapshot creation?

@sunilangadi2 , I hope you are aware of the normal snapshot creation with command - rbd snap create.., the second one is mentioned here - https://docs.ceph.com/en/quincy/rbd/rbd-mirroring/#create-image-mirror-snapshots

@aaSharma14 Thanks for the link, If image is already snapshot based mirrored what is the point of creating normal snapshots to it?

qa/tasks/mgr/dashboard/test_rbd.py Outdated Show resolved Hide resolved
@VasishtaShastry
Copy link
Contributor

@sunilangadi2 : hanks for the link, If image is already snapshot based mirrored what is the point of creating normal snapshots to it?

Regular image snapshot is a currently bigger feature which supports layering of images

@nizamial09
Copy link
Member

jenkins retest this please

@aaSharma14 aaSharma14 force-pushed the fix-rbd-snapshot-creation branch 2 times, most recently from 6d6f081 to 0deef5f Compare April 11, 2023 07:30
@nizamial09
Copy link
Member

nizamial09 commented Apr 11, 2023

lint failure

CHAR-0x1B][0m: [NON-XML-CHAR-0x1B][1;31mE501[NON-XML-CHAR-0x1B][0m line too long (102 > 100 characters)
1     E501 line too long (102 > 100 characters)
lint: exit 1 (1.60 seconds) /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr/dashboard> flake8 --config=tox.ini ../../../../qa/tasks/mgr/dashboard pid=3980861
  lint: FAIL code 1 (33.12=setup[28.14]+cmd[3.39,1.60] seconds)
  evaluation failed :( (33.22 seconds)

@aaSharma14 aaSharma14 force-pushed the fix-rbd-snapshot-creation branch 2 times, most recently from 55e3ec2 to 55f78bd Compare April 12, 2023 04:44
There are two types of snapshots that can be created on a snapshot based mirroring image - Normal Snapshot(same as journal based snapshot) and Nirror Image Snapshot. Till now Dashboard allowed only Mirror image snapshot, this PR intends to enable both the types

Signed-off-by: Aashish Sharma <aasharma@redhat.com>
@aaSharma14
Copy link
Contributor Author

jenkins test make check

@nizamial09 nizamial09 merged commit abff7f0 into ceph:main Apr 14, 2023
8 of 13 checks passed
@nizamial09 nizamial09 deleted the fix-rbd-snapshot-creation branch April 14, 2023 06:36
yuvalif pushed a commit to yuvalif/ceph that referenced this pull request Apr 30, 2023
…eation

mgr/dashboard: Fix rbd snapshot creation

Reviewed-by: VasishtaShastry <NOT@FOUND>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
Reviewed-by: sunilangadi2 <NOT@FOUND>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
5 participants