Add configuration parameters to kueue-viz backend (listen-port, logs, verbosity, etc...)

[akram]

What would you like to be added:
Add configuration parameters to kueue-viz backend (listen-port, logs, verbosity, etc...)

Why is this needed:
In the evenutality of grouping frontend and backend containers into the same pod, listen ports should be parametrizable.
Currently they are hardcoded both for frontend and backend to 8080; making it impossible to have frontend and backend in the same pod.

Completion requirements:
Have a parameter --listen=0.0.0.0:8181 to the backend and other startup parameters.

This enhancement requires the following artifacts:

• Design doc
• API change
• Docs update

The artifacts should be linked in subsequent comments.

[akram]

/help
/good-first-issue

[k8s-ci-robot]

@akram:
This request has been marked as suitable for new contributors.

Guidelines

Please ensure that the issue body includes answers to the following questions:

• Why are we solving this issue?
• To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
• Does this issue have zero to low barrier of entry?
• How can the assignee reach out to you for help?

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

/help
/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[SD-13]

Hey @akram, I am new to this project and want to work on this issue.

I have a few doubts -
what would be the default backend port? 8181?
where else we need to update to use parameter values?

• kueue/cmd/experimental/kueue-viz/backend/main.go

  Line 49 in f98a2f9

  if err := r.Run(":8080"); err != nil {

• the backend dockerfile
• docs

[vihaan-that]

Dear @akram I would like to know if the issue is still up, as I want to take my chance at solving it.

Thank you

[akram]

Hey @akram, I am new to this project and want to work on this issue.

I have a few doubts - what would be the default backend port? 8181? where else we need to update to use parameter values?

• kueue/cmd/experimental/kueue-viz/backend/main.go

  Line 49 in f98a2f9

  if err := r.Run(":8080"); err != nil {

• the backend dockerfile
• docs

hi @SD-13 ,

initially the default port was 8080 . You can keep this value as default or use 8181 . The thing is that, in the future we may have kueue-viz-frontend and kueue-viz-backend live in the same pod, so they must use different ports. frontend already uses 8080 as a default port. So, use 8181 by default for backend would allow to have no configuration.

[mimowo]

/kind dashboard

[lekaf974]

@akram I proposed a configuration based approach with my PR can you take a look and give feedback

[mimowo]

/reopen
As the issue seems to suggest more things should be configurable that just the port.
I will let @akram to close the issue when it can be considered done.