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

Patch for cacheonly setting #990

Merged
merged 3 commits into from Nov 29, 2017

Conversation

j-mracek
Copy link
Member

No description provided.

@j-mracek
Copy link
Member Author

@rh-atomic-bot try

@rh-atomic-bot
Copy link

⌛ Trying commit 6ffb8bf with merge c81f164...

rh-atomic-bot pushed a commit that referenced this pull request Nov 27, 2017
The formal code reacts only to --cacheonly option.

https://bugzilla.redhat.com/show_bug.cgi?id=1492036

Closes: #990
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 27, 2017
The option was defined twice in "dnf.conf".

Closes: #990
Approved by: <try>
@rh-atomic-bot
Copy link

☀️ Test successful - status-papr
State: approved= try=True

The option was defined twice in "dnf.conf".
Attribute repo._md_only_cached is set on different place during
_progress_demands().
@j-mracek
Copy link
Member Author

@rh-atomic-bot try

@rh-atomic-bot
Copy link

⌛ Trying commit d7d560d with merge d3505c5...

rh-atomic-bot pushed a commit that referenced this pull request Nov 28, 2017
The formal code reacts only to --cacheonly option.

https://bugzilla.redhat.com/show_bug.cgi?id=1492036

Closes: #990
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 28, 2017
The option was defined twice in "dnf.conf".

Closes: #990
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 28, 2017
Attribute repo._md_only_cached is set on different place during
_progress_demands().

Closes: #990
Approved by: <try>
@rh-atomic-bot
Copy link

☀️ Test successful - status-papr
State: approved= try=True

@j-mracek
Copy link
Member Author

There is a positive feedback from user for the PR (https://bugzilla.redhat.com/show_bug.cgi?id=1492036 Comment 71)

@m-blaha
Copy link
Member

m-blaha commented Nov 29, 2017

looks good.

@m-blaha
Copy link
Member

m-blaha commented Nov 29, 2017

@rh-atomic-bot r+

@rh-atomic-bot
Copy link

📌 Commit d7d560d has been approved by m-blaha

@m-blaha m-blaha merged commit aebbc22 into rpm-software-management:master Nov 29, 2017
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.

None yet

3 participants