Skip to content

chore(testing): Changes to support running devcontainer based tests on Mac Silicon#72

Merged
klugorosado merged 1 commit intofeature/v2/bootstrapVMWatchfrom
dev/dpoole/add-support-for-testing-on-mac-silicon
May 16, 2024
Merged

chore(testing): Changes to support running devcontainer based tests on Mac Silicon#72
klugorosado merged 1 commit intofeature/v2/bootstrapVMWatchfrom
dev/dpoole/add-support-for-testing-on-mac-silicon

Conversation

@dpoole73
Copy link
Contributor

I have been unable to run the integration tests locally since upgrading my laptop. I worked with kevin to figure out the issues and the tests are working now.

  1. changing to build the test container using no-cache mode since if you have an old bad version it would not get rebuilt.
  2. changing the devconatiner config to force running amd64 rather than arm64
  3. tweaking the scripts to handle the slightly different process names and ps output when running in this way.

now, the tests pass on mac

…n Mac Silicon

I have been unable to run the integration tests locally since upgrading my laptop.  I worked with kevin to figure out the issues and the tests are working now.

1. changing to build the test container using no-cache mode since if you have an old bad version it would not get rebuilt.
1. changing the devconatiner config to force running amd64 rather than arm64
1. tweaking the scripts to handle the slightly different process names and ps output when running in this way.

now, the tests pass on mac
Copy link
Collaborator

@klugorosado klugorosado left a comment

Choose a reason for hiding this comment

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

LGTM - Tested on local devbox with WSL

@klugorosado klugorosado merged commit b484db3 into feature/v2/bootstrapVMWatch May 16, 2024
@klugorosado klugorosado deleted the dev/dpoole/add-support-for-testing-on-mac-silicon branch May 16, 2024 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants