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

limit pr run to ubuntu latest X first and last version supported, add… #1992

Closed
wants to merge 2 commits into from

Conversation

avifenesh
Copy link
Collaborator

@avifenesh avifenesh commented Jul 21, 2024

Limited the CI to run on PR and PUSH to run just on Ubuntu-latest and latest and last version supported for engine and language version.
Added Schedule run daily, one hour in between each workflow, start from 3 AM, GMT+3.
Added trigger on label "Core changes" that will need team approval to run.
Added all NodeJS versions supported.

In this PR changes worked as expected.

@avifenesh avifenesh requested a review from a team as a code owner July 21, 2024 16:02
@avifenesh avifenesh force-pushed the CI/TrrigerByLabel branch 3 times, most recently from e57a2d6 to 98be872 Compare July 21, 2024 22:20
.github/workflows/java.yml Show resolved Hide resolved
.github/workflows/java.yml Outdated Show resolved Hide resolved
.github/workflows/java.yml Outdated Show resolved Hide resolved
.github/workflows/python.yml Show resolved Hide resolved
.github/workflows/python.yml Outdated Show resolved Hide resolved
.github/workflows/python.yml Outdated Show resolved Hide resolved
@Yury-Fridlyand Yury-Fridlyand added the CI/CD CI/CD related label Jul 22, 2024
@avifenesh avifenesh force-pushed the CI/TrrigerByLabel branch from 98be872 to a463403 Compare July 24, 2024 13:48
@avifenesh avifenesh requested a review from Yury-Fridlyand July 24, 2024 13:52
@avifenesh avifenesh force-pushed the CI/TrrigerByLabel branch 2 times, most recently from 217c36b to d3dd301 Compare July 24, 2024 13:55
@avifenesh avifenesh requested a review from barshaul July 24, 2024 14:12
@avifenesh avifenesh force-pushed the CI/TrrigerByLabel branch from d3dd301 to b2acddf Compare July 24, 2024 14:19
Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

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

Great job!
Few final steps left:

  1. Check and fix CI failures or rebase if it is already fixed in another PR(s).
  2. Eliminate errors and warnings (as much as possible) in CI reports ("Unexpected input(s) 'github-token'", "Node.js 16 actions are deprecated" and so on).
  3. Merge with changes from Python client CI: Split lint and testing in CI + add reporting. #1976 and Node client CI: Add reporting #1983.
  4. Update uploaded artifact names if needed.
  5. Add short instructions to the developer guides or create a new one with common info in the root of the repository.
  6. Label your PR to see how it works. I think you can un-label an re-label it back to re-run extra CI.

.github/workflows/java.yml Outdated Show resolved Hide resolved
.github/workflows/node.yml Show resolved Hide resolved
.github/workflows/node.yml Outdated Show resolved Hide resolved
.github/workflows/python.yml Outdated Show resolved Hide resolved
@avifenesh avifenesh force-pushed the CI/TrrigerByLabel branch 2 times, most recently from 5e01333 to ddd2ef6 Compare July 28, 2024 12:57
Signed-off-by: avifenesh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD CI/CD related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants