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

fsck: Only print "marking commit partial" once #1544

Closed
wants to merge 2 commits into from

Conversation

cgwalters
Copy link
Member

Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.

Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.
@jlebon
Copy link
Member

jlebon commented Apr 18, 2018

@rh-atomic-bot r+ 391f3ba

@rh-atomic-bot
Copy link

⌛ Testing commit 391f3ba with merge 86d1b90...

rh-atomic-bot pushed a commit that referenced this pull request Apr 18, 2018
Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.

Closes: #1544
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member

jlebon commented Apr 19, 2018

@rh-atomic-bot r+ 8dd04a1

@rh-atomic-bot
Copy link

⌛ Testing commit 8dd04a1 with merge c765c11...

rh-atomic-bot pushed a commit that referenced this pull request Apr 19, 2018
Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.

Closes: #1544
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member

jlebon commented Apr 19, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 8dd04a1 with merge 138a0e0...

rh-atomic-bot pushed a commit that referenced this pull request Apr 19, 2018
Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.

Closes: #1544
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member

jlebon commented Apr 19, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 8dd04a1 with merge f4b915c...

rh-atomic-bot pushed a commit that referenced this pull request Apr 19, 2018
Let's only print if the commit isn't already partial; this
addresses a spam of "marking commit partial" from fsck.

Closes: #1544
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@cgwalters
Copy link
Member Author

Moved into #1548

@cgwalters cgwalters closed this Apr 20, 2018
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.

3 participants