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

osc-staging: provide --cleanup option for unignore. #704

Merged
merged 1 commit into from Mar 3, 2017

Conversation

@jberry-suse
Copy link
Contributor

jberry-suse commented Mar 2, 2017

This would make sense to run under some bot credentials to auto-cleanup. I will go ahead and run again Leap 42.3 for the time being. Since the ignored_requests file is version controlled messages and such can always be recovered if some sort of mistake is made.

Here is the output for Factory:

Removing 459341 which was superseded 5 days ago                                                                                            
Removing 451216 which was revoked 33 days ago                                                                                              
Removing 448595 which was superseded 45 days ago
Removing 454181 which was superseded 17 days ago
Removing 454246 which was superseded 15 days ago
Removing 459240 which was superseded 5 days ago
Removing 448102 which was revoked 33 days ago
Removing 445239 which was superseded 29 days ago
Removing 445240 which was superseded 28 days ago
Unignoring 9 requests
@jberry-suse jberry-suse force-pushed the jberry-suse:unignore-cleanup branch from 85c04f1 to c6b37d6 Mar 2, 2017
@coveralls

This comment has been minimized.

Copy link

coveralls commented Mar 2, 2017

Coverage Status

Coverage decreased (-0.1%) to 45.327% when pulling c6b37d6 on jberry-suse:unignore-cleanup into 9d322ee on openSUSE:master.

@DimStar77

This comment has been minimized.

Copy link
Contributor

DimStar77 commented Mar 2, 2017

Feature sounds good

@jberry-suse

This comment has been minimized.

Copy link
Contributor Author

jberry-suse commented Mar 3, 2017

Alternatively, I could change it to look for: revoked, accepted, or superseded (final states) with no time delay. Possibly better, but was not sure if declined and not revoked issues would linger. That is likely a separate issue.

@lnussel lnussel merged commit a9fb871 into openSUSE:master Mar 3, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jberry-suse jberry-suse deleted the jberry-suse:unignore-cleanup branch Mar 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.