Skip to content

RHIDP-7360 added sealights steps - #1136

Merged
rm3l merged 5 commits into
redhat-developer:mainfrom
npotluri-rh:sealights-nightly
May 15, 2025
Merged

RHIDP-7360 added sealights steps#1136
rm3l merged 5 commits into
redhat-developer:mainfrom
npotluri-rh:sealights-nightly

Conversation

@npotluri-rh

Copy link
Copy Markdown
Contributor

Description

sealights integration specific to rhdh-operator nightly run

Which issue(s) does this PR fix or relate to

PR acceptance criteria

  • Tests
  • Documentation

How to test changes / Special notes to the reviewer

Before merge please add SEALIGHTS_AGENT_TOKEN to repo secrets.
Tested on my forked branch
nightly

if: ${{ matrix.branch == 'main' }} is added to sealights steps to run only on main for now and not any other release branches as it overwrites the records with latest run.

Comment thread .github/workflows/nightly.yaml Outdated
Comment thread .github/workflows/nightly.yaml Outdated
Comment thread .github/workflows/nightly.yaml Outdated
@npotluri-rh

Copy link
Copy Markdown
Contributor Author

/assign @rm3l @gazarenkov

Comment thread .github/workflows/nightly.yaml Outdated
Comment thread .github/workflows/nightly.yaml Outdated

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

Let's see what we can get from this. I'll trigger a manual nightly run after this is merged to test this integration.

@openshift-ci openshift-ci Bot added the lgtm label May 15, 2025
@openshift-ci

openshift-ci Bot commented May 15, 2025

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rm3l

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rm3l
rm3l merged commit 4f84334 into redhat-developer:main May 15, 2025
@rm3l

rm3l commented May 15, 2025

Copy link
Copy Markdown
Member

Let's see what we can get from this. I'll trigger a manual nightly run after this is merged to test this integration.

Nightly run triggered manually: https://github.com/redhat-developer/rhdh-operator/actions/runs/15050090648

Fortune-Ndlovu pushed a commit to Fortune-Ndlovu/rhdh-operator that referenced this pull request Sep 6, 2025
…veloper#1136)

* added sealights steps

* removes empty line and fixed seaLights for messages

* pinned to specific version

* Apply suggestions from code review

---------

Co-authored-by: Armel Soro <armel@rm3l.org>
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