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

Add a tool 'uplift_beta' to set a ni when a bug is fixed in nightly and is affecting beta #912

Merged
merged 2 commits into from Mar 9, 2020

Conversation

@calixteman
Copy link
Contributor

@calixteman calixteman commented Mar 2, 2020

No description provided.

@coveralls
Copy link

@coveralls coveralls commented Mar 2, 2020

Coverage Status

Coverage decreased (-0.3%) to 24.87% when pulling eeb2936 on calixteman:uplift into f8e3150 on mozilla:master.

auto_nag/scripts/uplift_beta.py Outdated Show resolved Hide resolved
auto_nag/scripts/uplift_beta.py Outdated Show resolved Hide resolved
return {bugid: info for bugid, info in bugs.items() if bugid in data}

def filter_by_regr(self, bugs):
# Filter the bugs which don't have any regression or where the regressions are all closed

This comment has been minimized.

@marco-c

marco-c Mar 3, 2020
Contributor

Maybe when there are regressions that were fixed, we should note that in the comment and remind them to uplift all patches

This comment has been minimized.

@calixteman

calixteman Mar 3, 2020
Author Contributor

I'm not sure about the wording

auto_nag/scripts/uplift_beta.py Outdated Show resolved Hide resolved
@marco-c
marco-c approved these changes Mar 3, 2020
"f2": self.status_beta,
"o2": "anyexact",
"v2": "affected",
# Changed before 2 days ago and not changed after 2 days ago

This comment has been minimized.

@jcristau

jcristau Mar 3, 2020
Contributor

Does "last resolved" not give you what you need here?

This comment has been minimized.

@calixteman

calixteman Mar 6, 2020
Author Contributor

I'd say that last_resolved and this way to check are pretty similar but different.
AFAIK, cf_last_resolved is the last time that resolution is no more --- or something like that and here we check the last change on the status flag for nightly.
Anyway, I don't care so much, finally it's up to you, so what's the best option according to your needs ?

@calixteman calixteman merged commit 8a93526 into mozilla:master Mar 9, 2020
1 of 2 checks passed
1 of 2 checks passed
coverage/coveralls Coverage decreased (-0.3%) to 24.87%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@calixteman calixteman deleted the calixteman:uplift branch Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants