Skip to content

Comments

Run serverless CI only on PRs targeting master#859

Merged
NickDris merged 3 commits intomasterfrom
serverless-tests-only-on-master
Sep 19, 2025
Merged

Run serverless CI only on PRs targeting master#859
NickDris merged 3 commits intomasterfrom
serverless-tests-only-on-master

Conversation

@NickDris
Copy link
Contributor

Test triggers for serverless rally-tracks are fired off whenever a PR is created against all branches without exception, which is redundant since serverless rally-tracks should be tested only against main Elasticsearch version.

This change filters the base branch of a PR to be 'master' so we can test only the PRs that target master branch.

@NickDris NickDris requested a review from gbanasiak September 17, 2025 09:04
Copy link
Contributor

@gbanasiak gbanasiak left a comment

Choose a reason for hiding this comment

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

This pipeline runs not only for PRs but also in a schedule. I would expect the condition to cover this case too, e.g. ... or build.source == "schedule".

Copy link
Contributor

@gbanasiak gbanasiak left a comment

Choose a reason for hiding this comment

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

Provided a comment about the condition earlier.

@gbanasiak
Copy link
Contributor

@elasticmachine update branch

@NickDris NickDris merged commit 4cf6c47 into master Sep 19, 2025
13 checks passed
@NickDris NickDris deleted the serverless-tests-only-on-master branch September 19, 2025 14:40
gbanasiak pushed a commit to gbanasiak/rally-tracks that referenced this pull request Dec 8, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs

(cherry picked from commit 4cf6c47)
@gbanasiak
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@esbenchmachine esbenchmachine added the backport pending Awaiting backport to stable release branch label Dec 9, 2025
@esbenchmachine
Copy link
Collaborator

@NickDris
A backport is pending for this PR. Please add all required vX.Y version labels.

  • If it is intended for the current Elasticsearch release version, apply the corresponding version label.
  • If it also supports past released versions, add those labels too.
  • If it only targets a future version, wait until that version label exists and then add it.
    (Each rally-tracks version label is created during the feature freeze of a new Elasticsearch branch).

Backporting entails:

  1. Ensure the correct version labels exist in this PR.
  2. Ensure backport PRs have backport label and are passing tests.
  3. Merge backport PRs (you can approve yourself and enable auto-merge).
  4. Remove backport pending label from this PR once all backport PRs are merged.

Thank you!

gbanasiak added a commit that referenced this pull request Dec 9, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs

(cherry picked from commit 4cf6c47)

Co-authored-by: Nick Dris <nick.dris@elastic.co>
@gbanasiak gbanasiak added v8.15 and removed backport pending Awaiting backport to stable release branch labels Dec 9, 2025
@esbenchmachine
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
8.19
9.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@esbenchmachine
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
8.19
9.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

esbenchmachine added a commit that referenced this pull request Dec 13, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs

(cherry picked from commit 4cf6c47)

Co-authored-by: Nick Dris <nick.dris@elastic.co>
NickDris added a commit that referenced this pull request Dec 13, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs

(cherry picked from commit 4cf6c47)

Co-authored-by: Nick Dris <nick.dris@elastic.co>
@NickDris NickDris added the v9.1 label Dec 18, 2025
@esbenchmachine
Copy link
Collaborator

💔 All backports failed

Status Branch Result
9.1 The branch "9.1" does not exist

Manual backport

To create the backport manually run:

backport --pr 859

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@NickDris NickDris added v9.1 and removed v9.1 labels Dec 18, 2025
@esbenchmachine
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

NickDris added a commit to NickDris/rally-tracks that referenced this pull request Dec 18, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs
NickDris added a commit to NickDris/rally-tracks that referenced this pull request Dec 18, 2025
Serverless CI tests will be now tested only in PRs targetting master branch, or on scheduled runs
NickDris added a commit that referenced this pull request Dec 20, 2025
* Tests tracks selectively based on PR changes (#858)
* CI builds ES revision from sources based on es-version file (#875)
* Drop Python 3.9 and introduce Python 3.13 in CI (#877)
* [ES-13188] Update GH macOS hosted runner image (#883)
* Address pytest deprecations (#911)
* CI determines Elasticsearch build arguments using unified job in gh workflow [stateful] (#925)
* Fix an error where ci arguments were not applied properly (#928)
* Fix an error where ci arguments were not applied properly
* Test the change
* Test failed retry with another fix
* Previous fix failed retry
* Revert es-version to current
* Keep the indents if if/else
* Add backport reminder and reorganize the backporting actions. (#889)
* Rename IT folders (#938)
* Fix pytest skip argument (#905)
* Backport reminders - fix missing default values in workflow (#947)
* Upgrade pip to 25.2
* Fix conflict in it_tracks
* Update deprecated gh actions
* Track changes for 8.19
* Fix test job which does not require the output of other jobs
* Add backport reminder and reorganize the backporting actions. (#889)
* Fix an error where ci arguments were not applied properly (#928)
* Reduce filtering scope in CI workflow (#908)
* Add pytest marker for cars
* Address merge conflicts
* Run serverless CI only on PRs targeting master (#859)
* Accept pragma risks for joins esql
* Make esql-full-text-functions a snapshot_only_challenge
* Set index template for ingest_mode: data_stream (#849)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants