Skip to content

Conversation

@umago
Copy link
Contributor

@umago umago commented May 28, 2025

Description

The recent change introduced by the commit
023076d broke the "run" target. Without this patch, starting the service via "pdm start" would fail with the following error:

$ pdm start
python src/lightspeed-stack.py "run"
[09:26:45] INFO Lightspeed stack startup
usage: lightspeed-stack.py [-h] [-v] [-d]
lightspeed-stack.py: error: unrecognized arguments: run
make: *** [Makefile:6: run] Error 2

Type of change

  • Refactor
  • New feature
  • [x ] Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • [ x] I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Just run "pdm start"

The recent change introduced by the commit
023076d broke the "run" target. Without
this patch, starting the service via "pdm start" would fail with the
following error:

$ pdm start
python src/lightspeed-stack.py "run"
[09:26:45] INFO     Lightspeed stack startup
usage: lightspeed-stack.py [-h] [-v] [-d]
lightspeed-stack.py: error: unrecognized arguments: run
make: *** [Makefile:6: run] Error 2

Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
Copy link
Contributor

@tisnik tisnik left a comment

Choose a reason for hiding this comment

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

ok

@tisnik tisnik merged commit 1084cd7 into lightspeed-core:main May 28, 2025
10 checks passed
@umago umago deleted the fix-make-run branch June 6, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants