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
core: Block MBS Image upload #476
Conversation
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
Suggestion use a validation similar to ovirt-engine/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java Line 619 in 513d016
And once transfer is supported (shouldn't be too hard to do) we can just enable it in Regarding commit message and PR description: both upload and download are blocked, so I'd rephrase it to "transfer" |
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Show resolved
Hide resolved
0dd0868
to
8366d64
Compare
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add Bug-Url:
to the commit message after Signed-off-by
, and also add Bug-Url:
to the PR description.
Signed-off-by
in the PR description IMHO is redundant, up to you.
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Show resolved
Hide resolved
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Show resolved
Hide resolved
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
/ost |
2 similar comments
/ost |
/ost |
/ost |
1 similar comment
/ost |
...bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
Outdated
Show resolved
Hide resolved
Transfering image into MBS will be block inside TransferDiskImageCommand validation Signed-off-by: Artiom Divak <adivak@redhat.com>
/ost |
4 similar comments
/ost |
/ost |
/ost |
/ost |
Uploading image into MBS will be block inside TransferDiskImageCommand validation
Signed-off-by: Artiom Divak adivak@redhat.com