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

Configure flutter/impeller with all our CI automation #97686

Closed
2 of 8 tasks
Hixie opened this issue Feb 3, 2022 · 13 comments
Closed
2 of 8 tasks

Configure flutter/impeller with all our CI automation #97686

Hixie opened this issue Feb 3, 2022 · 13 comments
Labels
engine flutter/engine repository. See also e: labels. P2 Important issues not at the top of the work list team-infra Owned by Infrastructure team

Comments

@Hixie
Copy link
Contributor

Hixie commented Feb 3, 2022

We just created flutter/impeller: https://github.com/flutter/impeller

We need to:

  • configure some sort of CI for this repo. @chinmaygarde, can you describe what kinds of tests you have and what infrastructure they will need?
  • tell the autocommit bot should monitor this repo
  • add a dashboard to show CI status
  • add a "luci-status" check on PRs to report tree greenness on this repo
  • configure this repo for scorecard security checking (.allstar should already be set up)
  • configure the "requires test" and autolabeling bot to know about this repo (@chinmaygarde can you describe the appropriate heuristics for telling if a PR in this repo has tests?)
  • add a test that verifies that all files have the right license header (can be copied from the framework repo) - @chinmaygarde you might be best placed to do this
  • make sure there is a LICENSE file in the root of the repo that matches the engine/LICENSE file exactly (@chinmaygarde will do this one)
@Hixie Hixie added team-infra Owned by Infrastructure team P2 labels Feb 3, 2022
@CaseyHillers
Copy link
Contributor

This would make a good use of the go/flutter-infra-office-hours (Google internal link). The tl;dr from the infra side is:

  1. Ensure the flutter-dashboard github app is installed for flutter/impeller
  2. Update Cocoon's configs to look at flutter/impeller (https://github.com/flutter/cocoon/blob/main/app_dart/lib/src/service/config.dart)
  3. Add a .ci.yaml and related roller for the repo to generate LUCI builders
  4. Add LUCI recipes (or we can create a generic recipe if needed where you can specify the tests in a yaml like we do for cocoon)

@godofredoc
Copy link
Contributor

  1. added flutter/impeller to the list of repositories enabled in the flutter-dashboard app. This is the only step that requires admin permissions to the repo.

  2. engine_v2 recipes should be the easiest way to build and test this repository

@bdero
Copy link
Member

bdero commented Feb 26, 2022

It looks like the scorecard check started failing in impeller starting on one of my commits yesterday, but I'm not sure how to diagnose the problem. Is there a different dashboard that has more diagnostic/actionable output? I'm not seeing any here: https://github.com/flutter/impeller/runs/5327601854?check_suite_focus=true

@Hixie
Copy link
Contributor Author

Hixie commented Feb 26, 2022

That's odd. Can you file a new infra bug for that?

@bdero
Copy link
Member

bdero commented Feb 26, 2022

Sure, done: #99185

@zanderso
Copy link
Member

Before we jump into this, I think we first need to decide whether to simply bring the impeller source into the Engine repo directly instead of having a separate repo. We might discuss other motivations, but it's only in a separate repo now to help us bootstrap.

@zanderso zanderso added the engine flutter/engine repository. See also e: labels. label Feb 27, 2022
@chinmaygarde chinmaygarde self-assigned this Feb 28, 2022
@chinmaygarde
Copy link
Member

I'll figure out the licenses stuff this week. In the engine sync last week, we discussed moving the repo entirely into the engine repo but decided against it at this time. Instead, re-evaluating the decision on a week by week basis.

@godofredoc
Copy link
Contributor

Sounds good, please let us know if it is decided to keep the repo to enable CI on it.

chinmaygarde added a commit to chinmaygarde/impeller that referenced this issue Mar 8, 2022
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
chinmaygarde added a commit to flutter/impeller that referenced this issue Mar 8, 2022
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
chinmaygarde added a commit to chinmaygarde/impeller that referenced this issue Mar 8, 2022
…lock.

Also fixes the files with missing licenses. This check is somewhat easy with
Impeller than in the engine because all source files must have the same license
block.

Resolves an action item in the umbrella issue flutter/flutter#97686
chinmaygarde added a commit to flutter/impeller that referenced this issue Mar 8, 2022
…lock. (#63)

Also fixes the files with missing licenses. This check is somewhat easy with
Impeller than in the engine because all source files must have the same license
block.

Resolves an action item in the umbrella issue flutter/flutter#97686.
@chinmaygarde
Copy link
Member

I'll ping this thread as we re-evaluate. For now, it is a separate repo. I've completed the action items related to licenses.

@chinmaygarde chinmaygarde removed their assignment Mar 9, 2022
@zanderso
Copy link
Member

At the meeting last week, we decided to re-evaluate again this week.

@godofredoc godofredoc added P2 Important issues not at the top of the work list and removed P2 labels Mar 22, 2022
@godofredoc
Copy link
Contributor

Moving to P4 until we get more clarity on whether we will be enabling CI on github/impeller or not.

dnfield pushed a commit to dnfield/engine that referenced this issue Apr 26, 2022
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
dnfield pushed a commit to dnfield/engine that referenced this issue Apr 26, 2022
…lock. (#63)

Also fixes the files with missing licenses. This check is somewhat easy with
Impeller than in the engine because all source files must have the same license
block.

Resolves an action item in the umbrella issue flutter/flutter#97686.
dnfield pushed a commit to flutter/engine that referenced this issue Apr 27, 2022
Matches the one in the engine exactly as described in the [umbrella issue](flutter/flutter#97686).
dnfield pushed a commit to flutter/engine that referenced this issue Apr 27, 2022
…lock. (#63)

Also fixes the files with missing licenses. This check is somewhat easy with
Impeller than in the engine because all source files must have the same license
block.

Resolves an action item in the umbrella issue flutter/flutter#97686.
@zanderso
Copy link
Member

This is obsolete. Impeller has moved into the Engine repo.

@github-actions
Copy link

github-actions bot commented Jun 1, 2022

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
engine flutter/engine repository. See also e: labels. P2 Important issues not at the top of the work list team-infra Owned by Infrastructure team
Projects
None yet
Development

No branches or pull requests

6 participants