Skip to content

Conversation

@LantaoJin
Copy link
Member

Description

MacOS build workflows were removed in #2662 because we don't have the M series artifacts. This PR addresses the artifact not found problem via replace macos-latest by macos-13.

macos-13 is the MacOS version used in workflows of OpenSearch Core:
Screenshot 2024-07-16 at 18 37 40

Issues Resolved

workflow failure

Check List

  • [-] New functionality includes testing.
    • [-] All tests pass, including unit test, integration test and doctest
  • [-] New functionality has been documented.
    • [-] New functionality has javadoc added
    • [-] New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Lantao Jin <ltjin@amazon.com>
@LantaoJin
Copy link
Member Author

@derek-ho @ykmr1224 @Swiddis could you take a look?

@derek-ho
Copy link
Collaborator

@LantaoJin thanks. My main reasoning for removing it is macos is not officially supported distribution of opensearch, so I don't see much point in running CI checks against it, and is throwing away github runner resources, but I won't block the PR/if OS core is doing it I don't see much harm in doing it as well.

@LantaoJin LantaoJin added the maintenance Improves code quality, but not the product label Jul 17, 2024
@penghuo penghuo merged commit 2c29a1a into opensearch-project:main Jul 17, 2024
LantaoJin added a commit that referenced this pull request Jul 17, 2024
Signed-off-by: Lantao Jin <ltjin@amazon.com>
manasvinibs pushed a commit to manasvinibs/sql that referenced this pull request Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Improves code quality, but not the product

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants