Skip to content

[RHIDP-13414] Update lightspeed core host#3059

Merged
Jdubrick merged 3 commits intoredhat-developer:mainfrom
Jdubrick:update-lightspeed-core-host
May 6, 2026
Merged

[RHIDP-13414] Update lightspeed core host#3059
Jdubrick merged 3 commits intoredhat-developer:mainfrom
Jdubrick:update-lightspeed-core-host

Conversation

@Jdubrick
Copy link
Copy Markdown
Contributor

@Jdubrick Jdubrick commented May 6, 2026

Hey, I just made a Pull Request!

✔️ Checklist

2 main changes:

  1. Refactors all references for the Lightspeed Core service to a constant value
  2. Changes the value from 0.0.0.0 to 127.0.0.1 to enforce same network (Pod)

Issue: https://redhat.atlassian.net/browse/RHIDP-13414

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

Jdubrick added 3 commits May 6, 2026 13:32
Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented May 6, 2026

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed-backend workspaces/lightspeed/plugins/lightspeed-backend patch v2.6.4

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.80%. Comparing base (3eec62c) to head (b3225a0).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3059   +/-   ##
=======================================
  Coverage   60.80%   60.80%           
=======================================
  Files        2067     2067           
  Lines       63895    63896    +1     
  Branches    16660    16660           
=======================================
+ Hits        38852    38853    +1     
  Misses      24481    24481           
  Partials      562      562           
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 3eec62c
ai-integrations 70.03% <ø> (ø) Carriedforward from 3eec62c
app-defaults 69.60% <ø> (ø) Carriedforward from 3eec62c
augment 69.36% <ø> (ø) Carriedforward from 3eec62c
bulk-import 72.44% <ø> (ø) Carriedforward from 3eec62c
cost-management 16.49% <ø> (ø) Carriedforward from 3eec62c
dcm 32.85% <ø> (ø) Carriedforward from 3eec62c
extensions 61.71% <ø> (ø) Carriedforward from 3eec62c
global-floating-action-button 73.75% <ø> (ø) Carriedforward from 3eec62c
global-header 61.68% <ø> (ø) Carriedforward from 3eec62c
homepage 50.84% <ø> (ø) Carriedforward from 3eec62c
konflux 91.01% <ø> (ø) Carriedforward from 3eec62c
lightspeed 69.96% <100.00%> (+<0.01%) ⬆️
mcp-integrations 81.59% <ø> (ø) Carriedforward from 3eec62c
orchestrator 33.15% <ø> (ø) Carriedforward from 3eec62c
quickstart 62.64% <ø> (ø) Carriedforward from 3eec62c
sandbox 79.56% <ø> (ø) Carriedforward from 3eec62c
scorecard 83.61% <ø> (ø) Carriedforward from 3eec62c
theme 64.54% <ø> (ø) Carriedforward from 3eec62c
translations 8.49% <ø> (ø) Carriedforward from 3eec62c
x2a 82.09% <ø> (ø) Carriedforward from 3eec62c

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3eec62c...b3225a0. Read the comment docs.

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

Copy link
Copy Markdown
Contributor

@yangcao77 yangcao77 left a comment

Choose a reason for hiding this comment

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

changes lgtm.
approved with assumption that verification has been performed against this change to ensure no regression.

@Jdubrick
Copy link
Copy Markdown
Contributor Author

Jdubrick commented May 6, 2026

changes lgtm. approved with assumption that verification has been performed against this change to ensure no regression.

@yangcao77 I was able to verify by running the plugin locally via Backstage and connecting to a Lightspeed Core instance. If for some reason in cluster it is behaving differently, the changes are setup in a way where we only need to change 1 line to flip back to 0.0.0.0 everywhere

@Jdubrick Jdubrick merged commit 96041ee into redhat-developer:main May 6, 2026
60 checks passed
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.

3 participants