Skip to content

Conversation

@zanderso
Copy link
Member

Relands #50579 with the following fixes:

  1. Assumes that any exception on the http socket means that the Azure metadata service is not available, meaning the tool is not running on Azure
  2. Additionally detects a bot when GITHUB_ACTIONS is defined in the environment.

@zanderso zanderso added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 12, 2020
@zanderso zanderso changed the title Reland detect azure Reland: [flutter_tools] Disable analytics for more bots Feb 12, 2020
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM. I can remove the BOT flag from my GitHub Actions script once this lands to verify (although that'll only tell us that at least one of the fixes works, not that both do).

@fluttergithubbot fluttergithubbot merged commit b9ecebf into flutter:master Feb 12, 2020
stuartmorgan-g added a commit to google/flutter-desktop-embedding that referenced this pull request Feb 12, 2020
As of flutter/flutter#50641 this should no longer be necessary.
@zanderso zanderso deleted the reland-detect-azure branch February 12, 2020 19:36
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants