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

Fix the development environment for linux, darwin and M1 arch #1041

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

prafull01
Copy link
Collaborator

@prafull01 prafull01 commented Aug 27, 2024

Update the code to support development on Darwin amd64 and arm64 architectures.

@prafull01 prafull01 force-pushed the development-environment branch 3 times, most recently from 77a34fb to 413fb38 Compare August 27, 2024 16:18
@prafull01 prafull01 marked this pull request as ready for review September 4, 2024 14:31
@prafull01 prafull01 force-pushed the development-environment branch 2 times, most recently from 059c867 to 5b19028 Compare September 8, 2024 11:06
@prafull01 prafull01 force-pushed the development-environment branch from 5b19028 to 0f651cc Compare October 18, 2024 13:51
@prafull01 prafull01 force-pushed the development-environment branch 7 times, most recently from c05e2d0 to 6538d7c Compare November 1, 2024 13:18
@prafull01 prafull01 force-pushed the development-environment branch from 6538d7c to 239607f Compare November 4, 2024 20:00
@prafull01 prafull01 requested a review from udnay November 4, 2024 20:00
Copy link

@udnay udnay left a comment

Choose a reason for hiding this comment

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

How can we avoid loading preflight for e2e tests?

sha256 = "a874d479f183f9e4c19a5c69b44955fabd2e250b467d2d9f0641ae91a82ddbea",
urls = ["https://storage.googleapis.com/cert-manager-testing-assets/kube-apiserver-1.17.3_darwin_amd64"],
sha256 = "800eedc293bcc72eb962df3e8006d7106790ca583a4ea5c107453bab3bf859b4",
urls = ["https://storage.googleapis.com/cockroach-operator-testing-assets/kube-apiserver-1.24.2_darwin_amd64"],
Copy link

Choose a reason for hiding this comment

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

We should update kube-apiserver_linux_amd64 as well.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed it. Also, gen_rule doesn't have any such condition. I might have to look deeper to move fetch_preflight to some other target which have platform specific option.

@prafull01 prafull01 force-pushed the development-environment branch 3 times, most recently from 8803436 to 7c67d14 Compare November 5, 2024 20:20
Copy link

@udnay udnay left a comment

Choose a reason for hiding this comment

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

lgtm

@prafull01
Copy link
Collaborator Author

Will merge when test passes

@prafull01 prafull01 force-pushed the development-environment branch 8 times, most recently from f4f12f1 to 0d10dfb Compare November 6, 2024 19:32
@prafull01 prafull01 force-pushed the development-environment branch 2 times, most recently from 51be942 to ed20896 Compare November 10, 2024 06:01
@prafull01 prafull01 force-pushed the development-environment branch 4 times, most recently from 36e752f to f157001 Compare November 12, 2024 18:10
@prafull01 prafull01 force-pushed the development-environment branch from f157001 to 4d8a6c9 Compare November 12, 2024 18:59
@udnay udnay merged commit 339275d into cockroachdb:master Nov 14, 2024
8 checks passed
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