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

Flutter master channel is broken on flutter/codelabs CI #160558

Closed
domesticmouse opened this issue Dec 19, 2024 · 10 comments · Fixed by #160574
Closed

Flutter master channel is broken on flutter/codelabs CI #160558

domesticmouse opened this issue Dec 19, 2024 · 10 comments · Fixed by #160574
Labels
monorepo monorepo related issues (infra, process, workflow)

Comments

@domesticmouse
Copy link
Contributor

I'm seeing the following behavior only on the master channel of my flutter/codelabs CI:

  dart --version
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    JAVA_HOME: /opt/hostedtoolcache/Java_Zulu_jdk/17.0.1[3](https://github.com/flutter/codelabs/actions/runs/12401981182/job/34622521206#step:5:3)-11/x64
    JAVA_HOME_17_X64: /opt/hostedtoolcache/Java_Zulu_jdk/17.0.13-11/x6[4](https://github.com/flutter/codelabs/actions/runs/12401981182/job/34622521206#step:5:4)
    FLUTTER_ROOT: /opt/hostedtoolcache/flutter/master-any-x64
    PUB_CACHE: /home/runner/.pub-cache
fatal: Not a valid object name upstream/master

Full context: https://github.com/flutter/codelabs/actions/runs/12401981182/job/34622521206

@domesticmouse domesticmouse added the monorepo monorepo related issues (infra, process, workflow) label Dec 19, 2024
@domesticmouse
Copy link
Contributor Author

You can see the various attempts to fix this breakage in flutter/codelabs#2273

@jtmcdole
Copy link
Contributor

fatal: Not a valid object name upstream/master

We calculate the engine hash based on the difference between upstream/master and HEAD. Is your upstream origin?

@domesticmouse
Copy link
Contributor Author

This is happening in CI, so it's subosito/flutter-action pulling a Flutter channel in the CI bot. That repo does checks for stable, beta, and master on linux, macos and windows hosts.

@jtmcdole
Copy link
Contributor

Yep; master doesn't work because it can't find the HASH for the engine because almost assuredly action is checking out with origin vs upstream

@domesticmouse
Copy link
Contributor Author

So subosito/flutter-action needs to be fixed?

@jtmcdole
Copy link
Contributor

No - I think we could fix this by testing for upstream vs origin:

git remote get-url upstream

I'm looking deeper.

@a-h-mzd
Copy link

a-h-mzd commented Dec 19, 2024

This also happens on local machines and is not exclusive to CI tools. #160552

@jtmcdole
Copy link
Contributor

We're aware and making a change

jtmcdole added a commit to jtmcdole/flutter that referenced this issue Dec 19, 2024
gnprice added a commit to gnprice/zulip-flutter that referenced this issue Dec 19, 2024
github-merge-queue bot pushed a commit that referenced this issue Dec 19, 2024
Several use cases for `origin` as default:

* Some github actions checkout the tree
* Some users download from flutter.dev and then use `master`

Fixes #160558
chrisbobbe pushed a commit to zulip/zulip-flutter that referenced this issue Dec 19, 2024
domesticmouse added a commit to flutter/codelabs that referenced this issue Dec 19, 2024
Land once flutter/flutter#160558 is fixed.

## Pre-launch Checklist

- [x] I read the [Effective Dart: Style] _recently_, and have followed
its advice.
- [x] I signed the [CLA].
- [x] I updated/added relevant documentation (doc comments with `///`).
- [ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-devrel
channel on [Discord].

<!-- Links -->
[Effective Dart: Style]:
https://dart.dev/guides/language/effective-dart/style
[CLA]: https://cla.developers.google.com/
[Discord]:
https://github.com/flutter/flutter/blob/master/docs/contributing/Chat.md
@jtmcdole
Copy link
Contributor

@a-h-mzd - can you confirm this has fixed your issue?

@a-h-mzd
Copy link

a-h-mzd commented Dec 20, 2024

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monorepo monorepo related issues (infra, process, workflow)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants