Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Application services 93.0.1 #12044

Merged
merged 3 commits into from
May 3, 2022

Conversation

bendk
Copy link
Contributor

@bendk bendk commented Apr 21, 2022

Note: the tabs constructor commit was approved in #11799

Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Changelog: This PR includes a changelog entry or does not need one
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

After merge

  • Milestone: Make sure issues closed by this pull request are added to the milestone of the version currently in development.
  • Breaking Changes: If this is a breaking change, please push a draft PR on Reference Browser to address the breaking issues.

@bendk
Copy link
Contributor Author

bendk commented Apr 21, 2022

Note: Sammy also opened #12029 which updates a-s to v91.1.2. This change contains the fix from that, plus an updated expiration date on the logins encryption key regeneration metrics, and many other changes.

I don't think any of the other changes are particularly risky, but there is a lot of them.

AFAIK, the only one that has any urgency is the logins encryption key metric expiration date update. It would be nice to get this merged to avoid the metrics expiring. But I don't think that it would be too bad if they expire, since I think they'll start working again once a-c does updates to v93.0.1. Missing a few weeks of metrics isn't a big deal.

So I guess it's a choice between:

  • Upgrading to v93.0.1 for the 102 release cycle.
  • Upgrading to v91.1.2 for the 102 release cycle and v93.0.1 for the 103 cycle

Copy link
Contributor

@Amejia481 Amejia481 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a minor nit on the commit messages "Incrment".

@Amejia481
Copy link
Contributor

👋🏽
@bendk just to confirm, the only critical patch is already on #12029? I'm asking as only have one week for this cycles to end.

@bendk
Copy link
Contributor Author

bendk commented Apr 22, 2022

👋🏽 @bendk just to confirm, the only critical patch is already on #12029? I'm asking as only have one week for this cycles to end.

Yes I think that's correct. AFAIK, the second most important patch is extending the logins encryption key metrics, but I would not call that critical.

@Amejia481
Copy link
Contributor

Amejia481 commented Apr 22, 2022

We already updated to v91.1.2 on main #12031, maybe as we don't have anything urgent on v93.0.1 and only one week left of this cycle. I think we could wait until the 102 release cycle.

@bendk bendk force-pushed the application-services-93.0.1 branch from 6921ff2 to e616a06 Compare May 2, 2022 22:03
@jonalmeida
Copy link
Contributor

We've cut the 101 branch, so we can land this now.

@jonalmeida jonalmeida closed this May 3, 2022
@jonalmeida jonalmeida reopened this May 3, 2022
@jonalmeida
Copy link
Contributor

Network intermittent?

[task 2022-05-02T22:37:16.886Z] * What went wrong:
[task 2022-05-02T22:37:16.886Z] A problem occurred configuring root project 'vcs'.
[task 2022-05-02T22:37:16.886Z] > Could not resolve all artifacts for configuration ':classpath'.
[task 2022-05-02T22:37:16.886Z]    > Could not find com.android.tools.build:gradle:7.0.0.
[task 2022-05-02T22:37:16.886Z]      Searched in the following locations:
[task 2022-05-02T22:37:16.886Z]        - http://localhost:8081/nexus/content/repositories/google/com/android/tools/build/gradle/7.0.0/gradle-7.0.0.pom
[task 2022-05-02T22:37:16.886Z]        - http://localhost:8081/nexus/content/repositories/central/com/android/tools/build/gradle/7.0.0/gradle-7.0.0.pom
[task 2022-05-02T22:37:16.886Z]        - https://plugins.gradle.org/m2/com/android/tools/build/gradle/7.0.0/gradle-7.0.0.pom
[task 2022-05-02T22:37:16.886Z]      Required by:
[task 2022-05-02T22:37:16.887Z]          project :

@jonalmeida jonalmeida added the 🛬 needs landing PRs that are ready to land label May 3, 2022
@jonalmeida
Copy link
Contributor

Ack, the changelog entry is in the old place, but I'll move it to the new one once this lands because it'll be a pain to restart CI again.

@mergify mergify bot merged commit 5513d43 into mozilla-mobile:main May 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛬 needs landing PRs that are ready to land
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants