Skip to content

pipeline: changes to support nightly toolchain #8161

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

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Dec 6, 2024

Make changes to ensure success of the self-hosted macOS pipeline that executes using the nightly toolchain.

Motivation:

When enabling the nightly toolchain build, the CI build does not set SDKROOT environment variable on Darwin, which causes some tests to fail

Modifications:

  1. Set SDKROOT in build-using-self when on Darwin. When using the nightly toolchain, the SDKROOT environment variable was not set. Explicitely set this to the default xcode installation only when on Darwin.
  2. Run the testTestsCanLinkAgainstAsyncExecutable only if it's executed on macOS 15 or never.
  3. Add additional logging to troubleshoot the bootstap failure and convert to use python's logging module instead of print statements.

Verifies #8137

@plemarquand
Copy link
Contributor

@swift-ci test

@plemarquand
Copy link
Contributor

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch 5 times, most recently from 0041735 to d128b12 Compare December 11, 2024 16:50
@plemarquand
Copy link
Contributor

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch 4 times, most recently from ca70f24 to 9d95883 Compare December 11, 2024 17:35
@bnbarham
Copy link
Contributor

@swift-ci please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from 9d95883 to 4f4ff6f Compare December 11, 2024 20:11
@rauhul
Copy link
Member

rauhul commented Dec 11, 2024

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from 4f4ff6f to 229ce09 Compare December 11, 2024 20:52
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 11, 2024

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from 229ce09 to 81606a8 Compare December 11, 2024 21:23
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 11, 2024

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from 81606a8 to ff887d9 Compare December 13, 2024 18:55
@shahmishal
Copy link
Member

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from ff887d9 to e387498 Compare December 17, 2024 13:34
@bkhouri bkhouri changed the title pipeline: Set SDKROOT in build-using-self when on Darwin pipeline: changes to support nightly toolchain Dec 17, 2024
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci please test

@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci Please test nightly self hosted

Make changes to ensure success of the self-hosted macOS pipeline that
executes using the nightly toolchain.

1/ Set SDKROOT in build-using-self when on Darwin

   When using the nighlty toolchain, the SDKROOT environment variable
   was not set.  Explicitely set this to the default xcode installation only
   when on Darwin.

2/ Run the testTestsCanLinkAgainstAsyncExecutable only if it's executed
   on macOS 15 or never.

3/ Add additional logging to troubleshoot the bootstap failure and
   convert to use python's logging module instead of print statements.
@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from e387498 to 5a56aa0 Compare December 17, 2024 13:42
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci Please test nightly self hosted

@bkhouri bkhouri force-pushed the t/main/update_test_to_support_nightly_toolchain branch from 5a56aa0 to 4b83823 Compare December 17, 2024 13:59
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci Please test nightly self hosted

@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci please test

@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 17, 2024

@swift-ci please test windows

set -x

env | sort
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this? I assume it was for debugging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Primarily informative information. I think it's valuable to have.

@shahmishal shahmishal merged commit b3798b2 into swiftlang:main Dec 18, 2024
6 checks passed
bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Feb 7, 2025
The `Utilities/bootstrap` script is no longer emitting Swift PM build
status information.

This change reverted the swiftlang#8161 and then converted the print calls to use
python's logging module.

Fixes swiftlang#8281
bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Feb 10, 2025
The `Utilities/bootstrap` script is no longer emitting Swift PM build
status information.

This change reverted the swiftlang#8161 and then converted the print calls to use
python's logging module.

Fixes swiftlang#8281
bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Feb 11, 2025
The `Utilities/bootstrap` script is no longer emitting Swift PM build
status information.

This change reverted the swiftlang#8161 and then converted the print calls to use
python's logging module.

Fixes swiftlang#8281
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.

6 participants