Skip to content

Commit

Permalink
chore: move to macstadium intel runner for circle darwinx64 [run ci] (#…
Browse files Browse the repository at this point in the history
…29300)

* chore: move to macstadium intel runner for circle darwinx64 [run ci]

* update cache and try runner machine version 3 [run ci]

* prevent stop only from failing (maybe we remove this?) [run ci]

* what happens to the dir after job [run ci]

* try this [run ci]

* retrigger x64 job mac [run ci]

* bump cache to verify functionality [run ci]

* bump cache key to trigger full dependency cache [run ci]

* rebump cache while user is not logged in [run ci]

* bump cache to check full rerun [run ci]
  • Loading branch information
AtofStryker authored Apr 16, 2024
1 parent 5a8618f commit 768afcc
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 32 deletions.
2 changes: 1 addition & 1 deletion .circleci/cache-version.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
# Bump this version to force CI to re-create the cache from scratch.

04-8-24
04-15-24-macstadium-3
48 changes: 21 additions & 27 deletions .circleci/workflows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ macWorkflowFilters: &darwin-workflow-filters
- equal: [ 'update-v8-snapshot-cache-on-develop', << pipeline.git.branch >> ]
- equal: [ 'cacie/dep/electron-27', << pipeline.git.branch >> ]
- equal: [ 'feat/protocol_shadow_dom_support', << pipeline.git.branch >> ]
- equal: [ 'ryanm/fix/service-worker-capture', << pipeline.git.branch >> ]
- equal: [ 'chore/move_off_circle_to_macstadium_amd64', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
Expand All @@ -58,7 +58,7 @@ linuxArm64WorkflowFilters: &linux-arm64-workflow-filters
- equal: [ 'update-v8-snapshot-cache-on-develop', << pipeline.git.branch >> ]
- equal: [ 'cacie/dep/electron-27', << pipeline.git.branch >> ]
- equal: [ 'feat/protocol_shadow_dom_support', << pipeline.git.branch >> ]
- equal: [ 'em/circle2', << pipeline.git.branch >> ]
- equal: [ 'chore/move_off_circle_to_macstadium_amd64', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
Expand All @@ -84,7 +84,7 @@ windowsWorkflowFilters: &windows-workflow-filters
- equal: [ 'cacie/dep/electron-27', << pipeline.git.branch >> ]
- equal: [ 'feat/protocol_shadow_dom_support', << pipeline.git.branch >> ]
- equal: [ 'lerna-optimize-tasks', << pipeline.git.branch >> ]
- equal: [ 'chore/reduce_windows_flake', << pipeline.git.branch >> ]
- equal: [ 'chore/move_off_circle_to_macstadium_amd64', << pipeline.git.branch >> ]
- matches:
pattern: /^release\/\d+\.\d+\.\d+$/
value: << pipeline.git.branch >>
Expand All @@ -111,11 +111,8 @@ executors:
# executor to run on Mac OS
# https://circleci.com/docs/2.0/executor-types/#using-macos
# https://circleci.com/docs/2.0/testing-ios/#supported-xcode-versions
mac:
macos:
# Executor should have Node >= required version
xcode: "14.0.1"
resource_class: macos.x86.medium.gen2
darwin-amd64:
machine: true
environment:
PLATFORM: darwin

Expand Down Expand Up @@ -154,7 +151,7 @@ commands:
name: Set environment variable to determine whether or not to persist artifacts
command: |
echo "Setting SHOULD_PERSIST_ARTIFACTS variable"
echo 'if ! [[ "$CIRCLE_BRANCH" != "develop" && "$CIRCLE_BRANCH" != "release/"* && "$CIRCLE_BRANCH" != "publish-binary" && "$CIRCLE_BRANCH" != "feat/protocol_shadow_dom_support" && "$CIRCLE_BRANCH" != "cacie/dep/electron-27" ]]; then
echo 'if ! [[ "$CIRCLE_BRANCH" != "develop" && "$CIRCLE_BRANCH" != "release/"* && "$CIRCLE_BRANCH" != "publish-binary" && "$CIRCLE_BRANCH" != "feat/protocol_shadow_dom_support" && "$CIRCLE_BRANCH" != "chore/move_off_circle_to_macstadium_amd64" ]]; then
export SHOULD_PERSIST_ARTIFACTS=true
fi' >> "$BASH_ENV"
# You must run `setup_should_persist_artifacts` command and be using bash before running this command
Expand Down Expand Up @@ -2262,6 +2259,7 @@ jobs:
resource_class:
type: string
default: medium+
resource_class: << parameters.resource_class >>
steps:
- restore_cached_workspace
- clone-repo-and-checkout-branch:
Expand Down Expand Up @@ -3364,15 +3362,15 @@ darwin-x64-workflow: &darwin-x64-workflow
jobs:
- node_modules_install:
name: darwin-x64-node-modules-install
executor: mac
resource_class: macos.x86.medium.gen2
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
only-cache-for-root-user: true

- build:
name: darwin-x64-build
context: test-runner:env-canary
executor: mac
resource_class: macos.x86.medium.gen2
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
requires:
- darwin-x64-node-modules-install

Expand All @@ -3383,33 +3381,29 @@ darwin-x64-workflow: &darwin-x64-workflow
- test-runner:upload
- test-runner:commit-status-checks
- test-runner:build-binary
executor: mac
resource_class: macos.x86.medium.gen2
requires:
- darwin-x64-build

- test-kitchensink:
name: darwin-x64-test-kitchensink
executor: mac
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
requires:
- darwin-x64-build

- v8-integration-tests:
name: darwin-x64-v8-integration-tests
executor: mac
resource_class: macos.x86.medium.gen2
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
requires:
- darwin-x64-build

- driver-integration-memory-tests:
name: darwin-x64-driver-integration-memory-tests
executor: mac
resource_class: macos.x86.medium.gen2
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
requires:
- darwin-x64-build

- server-unit-tests-cloud-environment:
name: darwin-x64-driver-server-unit-tests-cloud-environment
executor: mac
resource_class: macos.x86.medium.gen2
executor: darwin-amd64
resource_class: cypress-io/intel-macstadium
requires:
- darwin-x64-build

Expand Down
2 changes: 1 addition & 1 deletion packages/app/cypress/e2e/runner/cloud-debug-filter.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ describe('cloud debug test filtering', () => {

cy.get('.debug-dismiss').contains('2 / 4 tests').click().waitForSpecToFinish()

// suite.only is respected
// suite only is respected
cy.withCtx((ctx) => {
ctx.coreData.cloudProject.testsForRunResults = {
'cypress/e2e/skip-and-only.cy.js': ['t3', 's1 t4'],
Expand Down
2 changes: 1 addition & 1 deletion packages/proxy/test/unit/http/request-middleware.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ describe('http/request-middleware', () => {

expect(ctx.req.headers!['x-cypress-is-from-extra-target']).not.to.exist
expect(ctx.req.isFromExtraTarget).to.be.true
expect(ctx.onlyRunMiddleware).to.be.calledWith(['MaybeSetBasicAuthHeaders', 'SendRequestOutgoing'])
expect(ctx['onlyRunMiddleware']).to.be.calledWith(['MaybeSetBasicAuthHeaders', 'SendRequestOutgoing'])
})

it('when it does not exist, removes header and sets in on the req', async () => {
Expand Down
4 changes: 2 additions & 2 deletions packages/proxy/test/unit/http/response-middleware.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ describe('http/response-middleware', function () {
.then(() => {
expect(ctx.res.set).to.be.calledWith(headers)

expect(ctx.onlyRunMiddleware).to.be.calledWith([
expect(ctx['onlyRunMiddleware']).to.be.calledWith([
'AttachPlainTextStreamFn',
'PatchExpressSetHeader',
'MaybeSendRedirectToClient',
Expand All @@ -144,7 +144,7 @@ describe('http/response-middleware', function () {

return testMiddleware([FilterNonProxiedResponse], ctx)
.then(() => {
expect(ctx.onlyRunMiddleware).not.to.be.called
expect(ctx['onlyRunMiddleware']).not.to.be.called
})
})
})
Expand Down

5 comments on commit 768afcc

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on 768afcc Apr 16, 2024

Choose a reason for hiding this comment

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

Circle has built the linux arm64 version of the Test Runner.

Learn more about this pre-release build at https://on.cypress.io/advanced-installation#Install-pre-release-version

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/13.7.4/linux-arm64/develop-768afcc5ae6057e66d2b7542725403b8b3a9ddf5/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on 768afcc Apr 16, 2024

Choose a reason for hiding this comment

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

Circle has built the linux x64 version of the Test Runner.

Learn more about this pre-release build at https://on.cypress.io/advanced-installation#Install-pre-release-version

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/13.7.4/linux-x64/develop-768afcc5ae6057e66d2b7542725403b8b3a9ddf5/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on 768afcc Apr 16, 2024

Choose a reason for hiding this comment

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

Circle has built the darwin arm64 version of the Test Runner.

Learn more about this pre-release build at https://on.cypress.io/advanced-installation#Install-pre-release-version

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/13.7.4/darwin-arm64/develop-768afcc5ae6057e66d2b7542725403b8b3a9ddf5/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on 768afcc Apr 16, 2024

Choose a reason for hiding this comment

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

Circle has built the darwin x64 version of the Test Runner.

Learn more about this pre-release build at https://on.cypress.io/advanced-installation#Install-pre-release-version

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/13.7.4/darwin-x64/develop-768afcc5ae6057e66d2b7542725403b8b3a9ddf5/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on 768afcc Apr 16, 2024

Choose a reason for hiding this comment

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

Circle has built the win32 x64 version of the Test Runner.

Learn more about this pre-release build at https://on.cypress.io/advanced-installation#Install-pre-release-version

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/13.7.4/win32-x64/develop-768afcc5ae6057e66d2b7542725403b8b3a9ddf5/cypress.tgz

Please sign in to comment.