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

Issue stratis cli 1085 #1087

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mulkieran
Copy link
Member

@mulkieran mulkieran commented Jul 23, 2024

Closes #1085

@mulkieran mulkieran self-assigned this Jul 23, 2024
@mulkieran mulkieran force-pushed the issue_stratis-cli_1085 branch 4 times, most recently from fd63c05 to fe70404 Compare July 23, 2024 22:39
@mulkieran
Copy link
Member Author

/packit build

@mulkieran
Copy link
Member Author

Failing on coverage...

@mulkieran
Copy link
Member Author

/packit build

Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/stratis-storage-stratis-cli-1087
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@mulkieran
Copy link
Member Author

Will need to update man pages for change to filesystem listing, but that is about all.

Signed-off-by: mulhern <amulhern@redhat.com>
Just return the engine error if origin not set.
When trying to cancel a revert, and the revert has not been scheduled
because the filesystem has no origin, better to reject the cancel
action because of the missing origin, rather than because the merge
status will not be changed.

Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
The revised text for the "Snapshot origin" field now looks like this:

   Snapshot origin
       If this filesystem is a snapshot, its origin. If the filesystem is
       a snapshot, whether or not it is scheduled to replace its origin
       next time the pool is started.

Signed-off-by: mulhern <amulhern@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

Write code to set and read MergeRequested D-Bus property
1 participant