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

Fixes #26257 - Use pages on package/errata index to prevent t/o #640

Merged
merged 1 commit into from Apr 9, 2019
Merged

Fixes #26257 - Use pages on package/errata index to prevent t/o #640

merged 1 commit into from Apr 9, 2019

Conversation

chris1984
Copy link
Member

@chris1984 chris1984 commented Apr 8, 2019

Export not having timeout and showing correct:

hammer -d content-view version export --id 2 --export-dir /tmp

[DEBUG 2019-04-08T15:33:30 API] Params: {
    "repository_id" => 5,
              :page => 193,
          :per_page => 50
}
[DEBUG 2019-04-08T15:33:30 API] Headers: {
    :params => {
        "repository_id" => 5,
                  :page => 193,
              :per_page => 50
    }
}

[root@exportpatch tmp]# du -sh /tmp/export-timeout-2.tar 
45G	/tmp/export-timeout-2.tar

Import machine, importing correctly:

hammer -d content-view version import --organization-id 1 --export-tar /var/lib/pulp/katello-export/export-timeout-2.tar

                                 :vary => "Accept-Encoding",
                     :content_encoding => "gzip",
                       :content_length => "534",
                         :content_type => "application/json; charset=utf-8"
}
[.....................................................................................................................................................................................................................................................] [100%]

Screenshot of repos looking good:

RHEL - https://nimb.ws/YLe8pw

RHSCL - https://nimb.ws/xHC4yg

Screenshot of CV published and imported:

https://nimb.ws/P4sYQm

Client able to install screen package from timeout cv:

Installed:
  screen.x86_64 0:4.1.0-0.25.20120314git3c2946.el7                                                                                                                                                                                                            

Complete!

Repo-id      : rhel-7-server-rpms/7Server/x86_64
Repo-name    : Red Hat Enterprise Linux 7 Server (RPMs)
Repo-revision: 1554752590
Repo-updated : Mon Apr  8 19:43:10 2019
Repo-pkgs    : 23,918
Repo-size    : 34 G
Repo-baseurl : https://importpatch.toledo.satellite.lab.eng.rdu2.redhat.com/pulp/repos/Default_Organization/Library/timeout/content/dist/rhel/server/7/7Server/x86_64/os
Repo-expire  : 1 second(s) (last: Mon Apr  8 19:53:20 2019)
  Filter     : read-only:present
Repo-filename: /etc/yum.repos.d/redhat.repo

@theforeman-bot
Copy link

Issues: #26257

@chris1984 chris1984 changed the title Fixes #26257 - Use pages on package/errata index to prevent timeout Fixes #26257 - Use pages on package/errata index to prevent t/o Apr 8, 2019
@Katello Katello deleted a comment from theforeman-bot Apr 8, 2019
@Katello Katello deleted a comment from theforeman-bot Apr 8, 2019
@chris1984
Copy link
Member Author

chris1984 commented Apr 8, 2019

@jyejare here is the pr if you want to test it, I am working on fixing the tests but the code should work.

@chris1984
Copy link
Member Author

@akofink updated the loop on the repo hash to collect packages/errata to it is own method and moved it to the cv_helper. Let me know if you want me to keep it in the main file and I can move it back, rubocop disables are gone now.

Also if you have any insight into why the test main export test is failing I would appreciate it, I fixed the other 2 but this one is failing still. I am guessing it is due to the for loop I have going?

Copy link
Member

@akofink akofink left a comment

Choose a reason for hiding this comment

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

Looks much better, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants