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

- read gzipped metadata file by chunks could prevent python from MemoryError #707

Conversation

midnightercz
Copy link
Contributor

Sayli Karmarkar and others added 30 commits December 23, 2014 01:04
Add travis file to flake8 the code
apply style guide to pulp_rpm
This includes some flake8 fixups

Conflicts:
	plugins/pulp_rpm/plugins/importers/yum/sync.py
	plugins/test/unit/plugins/importers/yum/test_sync.py
Conflicts:
	plugins/pulp_rpm/plugins/distributors/yum/publish.py
Fix flake8 so that it does not run on rhel5 where it can not be installe...
barnabycourt and others added 25 commits April 17, 2015 11:14
Conflicts:
	plugins/pulp_rpm/plugins/importers/yum/sync.py
	plugins/test/unit/plugins/importers/yum/test_sync.py
Enable basic auth as an option when syncing repos
Raise the version to 2.8.0-0.1.alpha.
Use the test package to start the database connection, rather than
relying on test superclasses to do it. This way, only one database
connection will happen during the unit tests rather than one per test
class.

Additionally, mock all instances of alterations to the server config
during testing.

https://pulp.plan.io/issues/131

re pulp#131
re pulp#940
@pulpbot
Copy link
Member

pulpbot commented Aug 11, 2015

Can one of the admins verify this patch?

@midnightercz midnightercz force-pushed the read-gzipped-metadata-file-by-chunks_PR branch from 34191d6 to b47a71d Compare August 11, 2015 08:20
@barnabycourt
Copy link
Contributor

This looks like it was based of the master branch and the PR was made against 2.6-dev. If you really want this on the 2.6.x release stream please rebase against 2.6-dev and update this PR. Alternately, please close this PR and submit against master.

@asmacdo
Copy link
Contributor

asmacdo commented Sep 18, 2015

@midnightercz, closing this since it probably needs to be reopened against a different branch. Please feel free to reopen against master or rebase your changes as suggested by @barnabycourt.

@asmacdo asmacdo closed this Sep 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants