Skip to content

Conversation

@tvolkert
Copy link
Contributor

@tvolkert tvolkert commented Dec 4, 2019

Currently, the docker_builder shard on Cirrus has no name, so it show up as "main"

@tvolkert
Copy link
Contributor Author

tvolkert commented Dec 4, 2019

@fkorotkov

@fkorotkov
Copy link
Contributor

fkorotkov commented Dec 4, 2019

Yeah, that's the defat name. 🤷‍♂️ You can add name filed as well though.

@tvolkert
Copy link
Contributor Author

tvolkert commented Dec 4, 2019

Is this patch correct, then?

@fkorotkov
Copy link
Contributor

Correct, you can even make it like "Docker Build"

Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

cool

@christopherfujino christopherfujino changed the title Name the docker_builer shard Name the docker_builder shard Dec 4, 2019
@tvolkert
Copy link
Contributor Author

tvolkert commented Dec 5, 2019

@fkorotkov it looks like the deploy_gallery-macos shard has been pending for over 2 1/2 hours. Is that normal?

@christopherfujino
Copy link
Contributor

@tvolkert do you want to just comment out this phase with a TODO to be resolved by this issue #45351? This phase will only run on releases anyway, and we know it's currently failing.

@tvolkert
Copy link
Contributor Author

tvolkert commented Dec 5, 2019

Regardless of if it's failing, I'd like it to be named something obvious. "main" was confusing and had me looking in the Cirrus config file for a task named "main"

@fluttergithubbot fluttergithubbot merged commit 6d848eb into flutter:master Dec 5, 2019
@tvolkert tvolkert deleted the cirrus2 branch December 5, 2019 20:16
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants