Project

General

Profile

Bug #32421

hammer export fails with super large chunk size (change chunk-size-mb to gb)

Added by Partha Aji about 2 months ago. Updated 6 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Inter Server Sync
Target version:

Description

```
  1. hammer content-export complete library --organization-title "export-153" --chunk-size-mb 100000000000000
    [................................................................................] [100%]
    Error: [Errno 32] Broken pipe
    undefined method `first' for nil:NilClass
    ```
    Fails while
```
  1. hammer content-export complete library --organization-title "export-153" --chunk-size-mb 1000000000000
    [................................................................................] [100%]
    ```
    works.

The issue here is that the user may accidentally assume `--chunk-size-mb` and feed unusually large numbers.
We need katello to handle this invalid size error better and fail gracefully.

Associated revisions

Revision 8bf6159f (diff)
Added by Lucy Fu about 1 month ago

Fixes #32421 - Replace --chunk-size-mb with --chunk-size-gb for hammer content export

Revision fcca49db (diff)
Added by Lucy Fu about 1 month ago

Refs #32421 - Add tests for --chunk-size-gb

History

#2 Updated by Partha Aji about 2 months ago

Correction "The issue here is that the user may accidentally assume `--chunk-size-mb` is in bytes instead of megabytes"

#3 Updated by James Jeffers about 2 months ago

Partha, which version should this be targeted to?

#4 Updated by Partha Aji about 2 months ago

  • Target version set to Katello 4.2.0

Lets move it to 4.2. We can error out if the chunk size is > 1 petabyte

#5 Updated by Partha Aji about 2 months ago

  • Triaged changed from No to Yes

#7 Updated by Partha Aji about 2 months ago

  • Subject changed from hammer export fails with super large chunk size to hammer export fails with super large chunk size (change chunk-size-mb to gb)

#8 Updated by The Foreman Bot about 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9331 added

#9 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/803 added

#10 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases Katello 4.2.0 added

#11 Updated by Lucy Fu about 1 month ago

  • Status changed from Ready For Testing to Closed

#12 Updated by The Foreman Bot about 1 month ago

  • Assignee set to Chris Roberts
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/806 added

Also available in: Atom PDF