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

[SYSE-87] r5-lts: test-controller job decides test scope #5944

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

alephnull
Copy link
Contributor

The complete list of features for tyk:release-5-lts: releng, el7, plugin-compiler-fix-vendor.

tyk:release-5-lts needs to support RHEL7. RHEL7 support was dropped in 5.1 and then brought back in 5.2.1. This PR may not directly affect RHEL7 support but it may need testing on RHEL7 to make sure it still works. Release/upgrade-tests will perform some basic tests on supported OSes but this is no guarantee of everything being alright.
tyk:release-5-lts needs to be built without module support and the dependencies vendored for the sake of the plugin compiler. This is implemented in release.yml. Note that the plugin compiler itself is not part of the templates and is built by its own workflow, this PR is probably only about the gateway.
🤖 Beep, boop, this PR has been generated by automation. Changes should be limited to templated code. Where allowed, this PR will be automatically merged when the branch protection rules are satisfied. If you think this was made by mistake, or if merging this might cause any instabilities, please bring it up on Slack or log a jira in the TD project.

@alephnull alephnull requested a review from a team as a code owner January 17, 2024 05:19
@alephnull alephnull enabled auto-merge (squash) January 17, 2024 05:19
Copy link

sweep-ai bot commented Jan 17, 2024

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

This is an automated message generated by Sweep AI.

Copy link
Contributor

API Changes

no api changes detected

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit:
Triggered by: pull_request (@alephnull)
Execution page

@alephnull alephnull force-pushed the releng/release-5-lts branch 2 times, most recently from 975529c to dd7e517 Compare January 17, 2024 05:48
@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 975529c
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: dd7e517
Triggered by: pull_request (@alephnull)
Execution page

@alephnull alephnull force-pushed the releng/release-5-lts branch 4 times, most recently from 69565e2 to add31ab Compare January 17, 2024 06:35
@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: cb73391
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: da7ebcd
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 69565e2
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: add31ab
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 9a280b6
Triggered by: pull_request (@alephnull)
Execution page

@alephnull alephnull force-pushed the releng/release-5-lts branch 2 times, most recently from b6ad9e8 to e468011 Compare January 17, 2024 07:06
Copy link
Contributor

💥 CI tests failed 🙈

git-state

all ok

Please look at the run or in the Checks tab.

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 89e5fa9
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result - postgres15-sha256 env: success
Branch used: refs/heads/release-5-lts
Commit: da13d78 SYSE-87 r5-lts: test-controller job decides test scope (#5944)

The complete list of features for tyk:release-5-lts: releng, el7,
plugin-compiler-fix-vendor.

tyk:release-5-lts needs to support RHEL7. RHEL7 support was dropped in
5.1 and then brought back in 5.2.1. This PR may not directly affect
RHEL7 support but it may need testing on RHEL7 to make sure it still
works. Release/upgrade-tests will perform some basic tests on supported
OSes but this is no guarantee of everything being alright.
tyk:release-5-lts needs to be built without module support and the
dependencies vendored for the sake of the plugin compiler. This is
implemented in release.yml. Note that the plugin compiler itself is
not part of the templates and is built by its own workflow, this PR is
probably only about the gateway.
🤖 Beep, boop, this PR has been generated by
automation.
Changes should be limited to templated code. Where allowed, this PR will
be automatically merged when the branch protection rules are satisfied.
If you think this was made by mistake, or if merging this might cause
any instabilities, please bring it up on Slack or log a jira in the TD
project.

Co-authored-by: Gromit policy@gromit
Triggered by: push (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: c95e286
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result - mongo44-sha256 env: success
Branch used: refs/heads/release-5-lts
Commit: da13d78 SYSE-87 r5-lts: test-controller job decides test scope (#5944)

The complete list of features for tyk:release-5-lts: releng, el7,
plugin-compiler-fix-vendor.

tyk:release-5-lts needs to support RHEL7. RHEL7 support was dropped in
5.1 and then brought back in 5.2.1. This PR may not directly affect
RHEL7 support but it may need testing on RHEL7 to make sure it still
works. Release/upgrade-tests will perform some basic tests on supported
OSes but this is no guarantee of everything being alright.
tyk:release-5-lts needs to be built without module support and the
dependencies vendored for the sake of the plugin compiler. This is
implemented in release.yml. Note that the plugin compiler itself is
not part of the templates and is built by its own workflow, this PR is
probably only about the gateway.
🤖 Beep, boop, this PR has been generated by
automation.
Changes should be limited to templated code. Where allowed, this PR will
be automatically merged when the branch protection rules are satisfied.
If you think this was made by mistake, or if merging this might cause
any instabilities, please bring it up on Slack or log a jira in the TD
project.

Co-authored-by: Gromit policy@gromit
Triggered by: push (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: b6ad9e8
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: e468011
Triggered by: pull_request (@alephnull)
Execution page

@alephnull alephnull force-pushed the releng/release-5-lts branch 2 times, most recently from ea76c8f to 817d4f9 Compare January 17, 2024 09:02
@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: ea76c8f
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 817d4f9
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 17, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 3d32c87
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: d4f97a0
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 964d0b8
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: e351a16
Triggered by: pull_request (@alephnull)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 21de58b
Triggered by: pull_request (@asutosh)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result - postgres15-murmur64 env: success
Branch used: refs/tags/v5.0.11-alpha1
Commit: 21de58b
Triggered by: push (@asutosh)
Execution page

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result - mongo44-murmur64 env: success
Branch used: refs/tags/v5.0.11-alpha1
Commit: 21de58b
Triggered by: push (@asutosh)
Execution page

Copy link

sonarcloud bot commented Jan 18, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

11 New issues
0 Security Hotspots
84.0% Coverage on New Code
0.1% Duplication on New Code

See analysis details on SonarCloud

@buger
Copy link
Member

buger commented Jan 18, 2024

API tests result: failure 🚫
Branch used: refs/pull/5944/merge
Commit: 32b8af9
Triggered by: pull_request (@asutosh)
Execution page

@alephnull alephnull merged commit da13d78 into release-5-lts Jan 19, 2024
29 of 30 checks passed
@alephnull alephnull deleted the releng/release-5-lts branch January 19, 2024 05:59
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.

3 participants