Skip to content

ci: re-enable size tracking in integration tests#23515

Closed
alexeagle wants to merge 1 commit into
angular:masterfrom
alexeagle:track_payload_size
Closed

ci: re-enable size tracking in integration tests#23515
alexeagle wants to merge 1 commit into
angular:masterfrom
alexeagle:track_payload_size

Conversation

@alexeagle

Copy link
Copy Markdown
Contributor

Fixes #23376

@alexeagle
alexeagle requested a review from gkalpak April 23, 2018 23:05
@vicb vicb added area: build & ci Related the build and CI infrastructure of the project action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 24, 2018
Comment thread .circleci/README.md
https://circleci.com/gh/angular/angular/edit#advanced-settings)

We use this as a symmetric AES encryption key to encrypt tokens like
a GitHub token that enables publishing snapshots.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

OOC, why not have the GitHub- or Firebase tokens directly as secret vars on CircleCI, instead of KEY?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think this is so that we can update these tokens in the future, or add new ones, using only version control operations. Having one decryption key makes this as similar as possible to Travis (where they hold a private key)

Comment thread .circleci/config.yml
name: Set Angular build environment variables
command: |
echo "export PROJECT_ROOT=$(pwd)" >> $BASH_ENV
# CircleCI does not expose the commit range variable but we can easily

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"easily" 😛

Comment thread .circleci/config.yml
# CircleCI does not expose the commit range variable but we can easily
# get what we want from the compare URL which looks like
# https://github.com/angular/angular/compare/4361560...f0d1403
echo "export COMMIT_RANGE=${CIRCLE_COMPARE_URL##*/}" >> $BASH_ENV

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

$CIRCLE_COMPARE_URL seems to be empty btw 😁

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yeah this part is totally broken...

@ngbot

ngbot Bot commented May 4, 2018

Copy link
Copy Markdown

Hi @alexeagle! This PR has merge conflicts due to recent upstream merges.
Please help to unblock it by resolving these conflicts. Thanks!

gkalpak added a commit to gkalpak/angular that referenced this pull request Oct 31, 2018
@kara kara closed this in c40677a Nov 2, 2018
kara pushed a commit that referenced this pull request Nov 2, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer area: build & ci Related the build and CI infrastructure of the project cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

re-enable size tracking in integration tests

4 participants