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

[native] Remove old CircleCI jobs #24280

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

czentgr
Copy link
Contributor

@czentgr czentgr commented Dec 18, 2024

This removes the remaining CircleCI jobs from the repo so they won’t get run on new PRs.

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

This removes the remaining CircleCI jobs from the repo so they won’t get run
on new PRs.
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Dec 18, 2024
@prestodb-ci prestodb-ci requested review from a team, wanglinsong and pratyakshsharma and removed request for a team December 18, 2024 19:50
@czentgr czentgr requested a review from majetideepak December 18, 2024 19:50
@czentgr
Copy link
Contributor Author

czentgr commented Dec 18, 2024

@majetideepak FYI
@tdcmeehan This is the PR to remove the CircleCI jobs. Once the service is deactivated these are expected to fail and cause PR failures.
There is one more change we need to make to handle the builds after the PR has merged to match with what we did with the CircleCI previously. See #24279
The current run of the CI with Github actions looks pretty stable and gives the desired results. With the last PR we should also improve the build times for new PRs quite a bit - currently, each new PR has no initial cache but once we get the master branch builds this should improve.

Edit: there is a check CircleCI Pipeline that wants the configuration and fails if removed. This isn't coming from a Github action but maybe set in the settings somewhere in Github as an integration point?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
from:IBM PR from IBM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants