Skip to content

fix(lightspeed): LS service Configurable port - #1064

Merged
rohitkrai03 merged 2 commits into
redhat-developer:mainfrom
karthikjeeyar:configurable-port
Jul 2, 2025
Merged

fix(lightspeed): LS service Configurable port#1064
rohitkrai03 merged 2 commits into
redhat-developer:mainfrom
karthikjeeyar:configurable-port

Conversation

@karthikjeeyar

@karthikjeeyar karthikjeeyar commented Jun 26, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Fixes:

https://issues.redhat.com/browse/RHDHPAI-922

Added Port Configuration in app-config.yaml:

lightspeed:
  servicePort: ${SERVICE_PORT}
  servers:
    - id: ${SERVER_ID}
      url: ${SERVER_URL}
      token: ${SERVER_TOKEN}

✔️ Checklist

  • 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)

@rhdh-gh-app

rhdh-gh-app Bot commented Jun 26, 2025

Copy link
Copy Markdown

Changed Packages

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

maysunfaisal
maysunfaisal reviewed Jun 26, 2025
View reviewed changes
Comment thread workspaces/lightspeed/plugins/lightspeed-backend/README.md Outdated
@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

rohitkrai03
rohitkrai03 approved these changes Jul 2, 2025
View reviewed changes

@rohitkrai03 rohitkrai03 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jul 2, 2025
@rohitkrai03
rohitkrai03 merged commit d51643f into redhat-developer:main Jul 2, 2025
9 checks passed
mfrancisc pushed a commit to mfrancisc/rhdh-plugins that referenced this pull request Jul 9, 2025
@karthikjeeyar Devtools
fix(lightspeed): LS service Configurable port (redhat-developer#1064)
0468e6c 
* make LS service port configurable

* make LS service port configurable
elai-shalev pushed a commit to elai-shalev/rhdh-plugins that referenced this pull request Jan 21, 2026
@karthikjeeyar
fix(lightspeed): LS service Configurable port (redhat-developer#1064)
c9029ce 
* make LS service port configurable

* make LS service port configurable
@rhdh-qodo-merge rhdh-qodo-merge Bot mentioned this pull request Apr 22, 2026
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maysunfaisal maysunfaisal maysunfaisal left review comments
@rohitkrai03 rohitkrai03 rohitkrai03 approved these changes
@yangcao77 yangcao77 Awaiting requested review from yangcao77

Assignees

@rohitkrai03 rohitkrai03

Labels

lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@karthikjeeyar @rohitkrai03 @maysunfaisal