Skip to content

Create Integration Tile & Resources for SolarWinds #20487

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Create Integration Tile & Resources for SolarWinds #20487

wants to merge 15 commits into from
+162 −0

Conversation

fraseriver
Copy link

@fraseriver fraseriver commented Jun 11, 2025 •
edited
Loading

What does this PR do?

ECT-2835

This is a WIP PR to migrate SolarWinds from dogweb to have a v2 manifest & integration tile.

Motivation

Add solarwinds integration tile to integrations page.

Review checklist (to be filled by reviewers)

  • Go to staging and check out the SolarWinds integration tile
  • Particularly review the v2 manifest & README and add any comments/changes

@fraseriver fraseriver requested review from a team as code owners June 11, 2025 19:54
rtrieu
rtrieu previously approved these changes Jun 11, 2025
View reviewed changes
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Some feedback for style consistency.

@temporal-github-worker-1 temporal-github-worker-1 bot dismissed rtrieu’s stale review June 11, 2025 22:06

Review from rtrieu is dismissed. Related teams and files:

  • documentation
    • solarwinds/README.md
@fraseriver fraseriver requested a review from a team as a code owner June 11, 2025 22:31
@fraseriver
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Jun 16, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-06-16 20:54:49 UTC ℹ️ Start processing command /merge

2025-06-16 20:55:00 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-06-17 00:55:05 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@fraseriver fraseriver removed the request for review from rtrieu June 17, 2025 16:00
@fraseriver
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Jun 17, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-06-17 16:00:48 UTC ℹ️ Start processing command /merge

2025-06-17 16:00:59 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-06-17 17:43:42 UTC ⚠️ MergeQueue: This merge request was unqueued

[email protected] unqueued this merge request

@temporal-github-worker-1 temporal-github-worker-1 bot dismissed stale reviews from apiazza-dd and steveny91 June 17, 2025 16:01

Review from apiazza-dd is dismissed. Related teams and files:

  • saas-integrations
    • .github/workflows/config/labeler.yml

Review from steveny91 is dismissed. Related teams and files:

  • agent-integrations
    • .github/workflows/config/labeler.yml
@fraseriver
Copy link
Author

/merge -c

@dd-devflow
Copy link

dd-devflow bot commented Jun 17, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-06-17 17:43:35 UTC ℹ️ Start processing command /merge -c

@fraseriver fraseriver requested a review from steveny91 June 17, 2025 19:27
@fraseriver fraseriver requested a review from apiazza-dd June 17, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steveny91 steveny91 steveny91 approved these changes

@urseberry urseberry urseberry approved these changes

@apiazza-dd apiazza-dd apiazza-dd approved these changes

@rtrieu rtrieu rtrieu left review comments

Assignees
No one assigned
Labels
agent/approved dev/testing dev/tooling docs/approved documentation ecosystems/review-requested integration/solarwinds mergequeue-status: removed product/review-requested qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/documentation team/saas-integrations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@fraseriver @steveny91 @urseberry @rtrieu @apiazza-dd