Project

General

Profile

Actions

Bug #32421

closed

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

Added by Partha Aji over 3 years ago. Updated over 2 years 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.

Actions #1

Updated by Partha Aji over 3 years ago

Actions #2

Updated by Partha Aji over 3 years ago

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

Actions #3

Updated by James Jeffers over 3 years ago

Partha, which version should this be targeted to?

Actions #4

Updated by Partha Aji over 3 years 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

Actions #5

Updated by Partha Aji over 3 years ago

  • Triaged changed from No to Yes
Actions #7

Updated by Partha Aji over 3 years 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)
Actions #8

Updated by The Foreman Bot over 3 years ago

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

Updated by The Foreman Bot over 3 years ago

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

Updated by The Foreman Bot over 3 years ago

  • Fixed in Releases Katello 4.2.0 added
Actions #11

Updated by Lucy Fu over 3 years ago

  • Status changed from Ready For Testing to Closed
Actions #12

Updated by The Foreman Bot over 3 years ago

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

Updated by The Foreman Bot over 2 years ago

  • Pull request https://github.com/Katello/katello/pull/10108 added
Actions

Also available in: Atom PDF