Skip to content

Experiments are enabled by default #5073

Closed

Description

@kenzieschmoll

This is happening in the released version of DevTools. Marking as P0 as we need to get this resolved and see if a cherry pick is possible.
@polina-c

Activity

  1. bkonyi commented on Jan 19, 2023

    @bkonyi
    Contributor

    FYI @itsjustkevin, we have some major breakages that made it into the DevTools shipped with the SDK.

  2. polina-c commented on Jan 19, 2023

    @polina-c
    Contributor

    kReleaseMode is false for some reasons in the build created by dart ./tool/build_e2e.dart

    so this line enables flags:

    if (!kReleaseMode) setExperimentsEnabled();
    

    This line does not exist in current version, so we are safe with our release process.

  3. itsjustkevin commented on Jan 19, 2023

    @itsjustkevin

    Thanks for calling this out @kenzieschmoll. I’m assuming in this situation, we need a new dart build and not a new flutter build? I’m still not versed around the devtools release process.

  4. bkonyi commented on Jan 19, 2023

    @bkonyi
    Contributor

    We'll need both a Flutter and Dart SDK release since DevTools is shipped as part of the Dart SDK.

  5. itsjustkevin commented on Jan 19, 2023

    @itsjustkevin
  6. XilaiZhang commented on Jan 19, 2023

    @XilaiZhang

    thanks for the info! is the bad build also included in the dart version that we used for flutter beta hotfix?

  7. kenzieschmoll commented on Jan 19, 2023

    @kenzieschmoll
    MemberAuthor

    Yes this is on every flutter version starting with 3.7.0-1.1.pre

  8. XilaiZhang commented on Jan 19, 2023

    @XilaiZhang

    thanks for explaining! @itsjustkevin in terms of flutter beta hotfix, shall we keep moving forward with the remaining steps, or shall we wait to grab a new dart version, engine cp etc.?

  9. itsjustkevin commented on Jan 20, 2023

    @itsjustkevin

    I think the safest bet here is to push a release with the newest dart hash. I may be wrong here, opinions on the matter @godofredoc?

  10. godofredoc commented on Jan 20, 2023

    @godofredoc
    Contributor

    What would be the impact if we push the beta release as expected and release a hotfix early next week?

  11. kenzieschmoll commented on Jan 20, 2023

    @kenzieschmoll
    MemberAuthor

    The risk would be that the DevTools Performance page is broken and our analytics are turned off until the hotfix goes out.

  12. kenzieschmoll commented on Jan 25, 2023

    @kenzieschmoll
    MemberAuthor

    This is resolved. The 2.20.1 cherrypick made it into the stable release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions