Skip to content

ROX-30751: Remove test kernel support available api#17826

Merged
JoukoVirtanen merged 2 commits intomasterfrom
jv-remove-TestKernelSupportAvailableApi
Nov 16, 2025
Merged

ROX-30751: Remove test kernel support available api#17826
JoukoVirtanen merged 2 commits intomasterfrom
jv-remove-TestKernelSupportAvailableApi

Conversation

@JoukoVirtanen
Copy link
Copy Markdown
Contributor

@JoukoVirtanen JoukoVirtanen commented Nov 15, 2025

Description

Kernel collection method has been removed, so this test should be removed as well.

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

CI is sufficient.

@rhacs-bot
Copy link
Copy Markdown
Contributor

Images are ready for the commit at e15a0cf.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.10.x-342-ge15a0cf3e6.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.14%. Comparing base (f014877) to head (e15a0cf).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #17826      +/-   ##
==========================================
- Coverage   49.15%   49.14%   -0.01%     
==========================================
  Files        2731     2731              
  Lines      201005   201005              
==========================================
- Hits        98800    98792       -8     
- Misses      94475    94479       +4     
- Partials     7730     7734       +4     
Flag Coverage Δ
go-unit-tests 49.14% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JoukoVirtanen JoukoVirtanen merged commit a091315 into master Nov 16, 2025
91 checks passed
@JoukoVirtanen JoukoVirtanen deleted the jv-remove-TestKernelSupportAvailableApi branch November 16, 2025 17:31
@dashrews78 dashrews78 added backport release-4.8 backport release-4.9 https://spaces.redhat.com/spaces/StackRox/pages/558727298 labels Nov 17, 2025
@rhacs-bot
Copy link
Copy Markdown
Contributor

The backport to release-4.9 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-4.9 release-4.9
# Navigate to the new working tree
cd .worktrees/backport-release-4.9
# Create a new branch
git switch --create backport-17826-to-release-4.9
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 a0913151d20ea14d4bbb5814e8649722303148e5
# Push it to GitHub
git push --set-upstream origin backport-17826-to-release-4.9
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-4.9

Then, create a pull request where the base branch is release-4.9 and the compare/head branch is backport-17826-to-release-4.9.

@rhacs-bot
Copy link
Copy Markdown
Contributor

The backport to release-4.8 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-4.8 release-4.8
# Navigate to the new working tree
cd .worktrees/backport-release-4.8
# Create a new branch
git switch --create backport-17826-to-release-4.8
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 a0913151d20ea14d4bbb5814e8649722303148e5
# Push it to GitHub
git push --set-upstream origin backport-17826-to-release-4.8
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-4.8

Then, create a pull request where the base branch is release-4.8 and the compare/head branch is backport-17826-to-release-4.8.

@dashrews78 dashrews78 mentioned this pull request Nov 17, 2025
9 tasks
@janisz janisz changed the title chore(test): Remove test kernel support available api ROX-30751: Remove test kernel support available api Nov 17, 2025
@janisz
Copy link
Copy Markdown
Contributor

janisz commented Nov 17, 2025

Since we remove the test should we also remove the deprecated service that was tested?

@dashrews78
Copy link
Copy Markdown
Contributor

Since we remove the test should we also remove the deprecated service that was tested?

+1 to that.

Molter73 added a commit that referenced this pull request Mar 24, 2026
Co-authored-by: Jouko Virtanen <jvirtane@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release-4.8 backport release-4.9 https://spaces.redhat.com/spaces/StackRox/pages/558727298

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants