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

Pin Windows CI to VS 17.9.7 toolset as runner image update has build failures #1611

Merged
merged 3 commits into from
Jun 10, 2024

Conversation

muhammadhamzasajjad
Copy link
Collaborator

@muhammadhamzasajjad muhammadhamzasajjad commented Jun 10, 2024

The new GH runner release 20240603.1 requires minimum 14.40 for MSVC toolset.

Set VS2022 BuildTools to 17.9.7 as version 17.10 is busted with several dependencies.

What does this implement or fix?

Any other comments?

Checklist

Checklist for code changes...
  • Have you updated the relevant docstrings, documentation and copyright notice?
  • Is this contribution tested against all ArcticDB's features?
  • Do all exceptions introduced raise appropriate error messages?
  • Are API changes highlighted in the PR description?
  • Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

@muhammadhamzasajjad muhammadhamzasajjad self-assigned this Jun 10, 2024
Copy link
Collaborator

@vasil-pashov vasil-pashov left a comment

Choose a reason for hiding this comment

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

LGTM given all tests are passing.

@muhammadhamzasajjad muhammadhamzasajjad marked this pull request as ready for review June 10, 2024 16:09
@muhammadhamzasajjad muhammadhamzasajjad changed the title Update MSVC toolset as per runner image update Pin Windows CI to VS 17.9.7 toolset as runner image update has build failures Jun 10, 2024
@muhammadhamzasajjad muhammadhamzasajjad merged commit 911a55c into master Jun 10, 2024
111 checks passed
@muhammadhamzasajjad muhammadhamzasajjad deleted the msvc-toolset-CI branch June 10, 2024 16:28
muhammadhamzasajjad added a commit that referenced this pull request Jun 18, 2024
…failures (#1611)

The new GH runner release
[20240603.1](https://github.com/actions/runner-images/releases/tag/win22%2F20240603.1)
requires minimum 14.40 for MSVC toolset.

Set [VS2022 BuildTools to
17.9.7](actions/runner-images#10004 (comment))
as version 17.10 is busted with several dependencies.

#### What does this implement or fix?

#### Any other comments?

#### Checklist

<details>
  <summary>
   Checklist for code changes...
  </summary>
 
- [ ] Have you updated the relevant docstrings, documentation and
copyright notice?
- [ ] Is this contribution tested against [all ArcticDB's
features](../docs/mkdocs/docs/technical/contributing.md)?
- [ ] Do all exceptions introduced raise appropriate [error
messages](https://docs.arcticdb.io/error_messages/)?
 - [ ] Are API changes highlighted in the PR description?
- [ ] Is the PR labelled as enhancement or bug so it appears in
autogenerated release notes?
</details>

<!--
Thanks for contributing a Pull Request to ArcticDB! Please ensure you
have taken a look at:
- ArcticDB's Code of Conduct:
https://github.com/man-group/ArcticDB/blob/master/CODE_OF_CONDUCT.md
- ArcticDB's Contribution Licensing:
https://github.com/man-group/ArcticDB/blob/master/docs/mkdocs/docs/technical/contributing.md#contribution-licensing
-->
grusev pushed a commit that referenced this pull request Nov 25, 2024
…failures (#1611)

The new GH runner release
[20240603.1](https://github.com/actions/runner-images/releases/tag/win22%2F20240603.1)
requires minimum 14.40 for MSVC toolset.

Set [VS2022 BuildTools to
17.9.7](actions/runner-images#10004 (comment))
as version 17.10 is busted with several dependencies.

#### What does this implement or fix?

#### Any other comments?

#### Checklist

<details>
  <summary>
   Checklist for code changes...
  </summary>
 
- [ ] Have you updated the relevant docstrings, documentation and
copyright notice?
- [ ] Is this contribution tested against [all ArcticDB's
features](../docs/mkdocs/docs/technical/contributing.md)?
- [ ] Do all exceptions introduced raise appropriate [error
messages](https://docs.arcticdb.io/error_messages/)?
 - [ ] Are API changes highlighted in the PR description?
- [ ] Is the PR labelled as enhancement or bug so it appears in
autogenerated release notes?
</details>

<!--
Thanks for contributing a Pull Request to ArcticDB! Please ensure you
have taken a look at:
- ArcticDB's Code of Conduct:
https://github.com/man-group/ArcticDB/blob/master/CODE_OF_CONDUCT.md
- ArcticDB's Contribution Licensing:
https://github.com/man-group/ArcticDB/blob/master/docs/mkdocs/docs/technical/contributing.md#contribution-licensing
-->
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.

3 participants