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

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

Merged
merged 10 commits into from
Apr 16, 2024
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
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'])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I originally fixed this due to cache issues on the mac machines, but then also realized that the stop-only script fails in local development in spots where it shouldn't. So I think these changes are warranted as the stop-only shouldn't be throwing for these as they are .only mocha tests.

})

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
Loading