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

(maint) Remove github macos tests #3905

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Conversation

rbrw
Copy link
Contributor

@rbrw rbrw commented Nov 1, 2023

These have been causing intermittent, and not so intermittent failures that appear to be due to extremely poor performance from the gihub runners, corroroborated by some online reports which also noted that they're expensive (per minute?).

Given that and the fact that no one's actively developing puppetdb from macos right now, remove the tests with the expectation that if/when we restore them, we'll restore them elsewhere.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

These have been causing intermittent, and not so intermittent failures
that appear to be due to extremely poor performance from the gihub
runners, corroroborated by some online reports which also noted that
they're expensive (per minute?).

Given that and the fact that no one's actively developing puppetdb
from macos right now, remove the tests with the expectation that
if/when we restore them, we'll restore them elsewhere.
@rbrw rbrw force-pushed the maint-drop-macos-tests branch from 63083b6 to ee0277f Compare November 7, 2023 00:17
@rbrw rbrw marked this pull request as ready for review November 7, 2023 00:17
@rbrw rbrw requested review from a team as code owners November 7, 2023 00:17
@austb austb merged commit bcdff76 into puppetlabs:7.x Nov 7, 2023
6 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.

4 participants