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

CIS Benchmark's Scan detail page config tab fails to show schedule info when scan is pending #12429

Open
richard-cox opened this issue Oct 30, 2024 · 0 comments
Labels
kind/bug QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this QA/manual-test Indicates issue requires manually testing
Milestone

Comments

@richard-cox
Copy link
Member

Setup

  • Rancher version: 2.9 (at least)

Describe the bug

  • cis scan detail page config tab should show schedule info regardless of running state of scan

To Reproduce

  • install cis benchmark
  • create a scan
    • supply schedule
  • kick off a scan
  • navigate quickly to detail page config tab (or ensure scan stays in pending state given queue of scans)

Result

  • no schedule info is shown

Expected Result

Screenshots

Additional context

  • caused by
    canBeScheduled() {
    // check if scan was created and run with an older cis install that doesn't support scheduling/alerting/warn state
    if (this.mode === _VIEW) {
    const warn = get(this.value, 'status.summary.warn');
    return !!warn || warn === 0;
    }
    return this.value.canBeScheduled();
    • warn is undefined, therefore canBeScheduled is false, so radio button and schedule settings not shown
    • probably can remove this.mode if block ?
  • detail page doesn't do anything fancy, just looks at value.canBeScheduled -
    canBeScheduled() {
    return this.value.canBeScheduled();
    },
@richard-cox richard-cox added this to the v2.11.0 milestone Oct 30, 2024
@github-actions github-actions bot added QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this and removed area/cis-benchmark labels Oct 30, 2024
@nwmac nwmac modified the milestones: v2.12.0, v2.11.0 Nov 1, 2024
@nwmac nwmac added the QA/manual-test Indicates issue requires manually testing label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this QA/manual-test Indicates issue requires manually testing
Projects
None yet
Development

No branches or pull requests

2 participants