This fixes the communication of the Web UI component with Marquez AP…

[thomas-delrue]

…I service in the Helm Chart

Problem

Deploying the current Helm Chart in main results in a broken Marquez Web Component because of a misconfiguration in the Helm Chart. In the container spec in the Web deployment.yaml file two environment variables are set, i.e. MARQUEZ_HOST and MARQUEZ_PORT, for the instance to be able to communicate with the Marquez API service in the K8s cluster.

The MARQUEZ_PORT is erroneously set to 5000, .Values.marquez.port, the port number used internally in the container for the API, but the Marquez API service is exposed via port number 80, .Values.service.port, within the K8s cluster

Solution

Small fix: define the value for MARQUEZ_PORT as .Values.service.port, i.e. 80. That way Marquez Web component can communicate via http://marquez:80 with the API backend.

Checklist

• You've signed-off your work
• Your changes are accompanied by tests (if relevant)
• Your change contains a small diff and is self-contained
• You've updated any relevant documentation (if relevant)
• You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
• You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
• You've included a header in any source code files (if relevant)

[boring-cyborg]

Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md).

[codecov]

Codecov Report

Merging #2430 (3b21698) into main (79a9c4f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2430   +/-   ##
=========================================
  Coverage     83.61%   83.61%           
  Complexity     1214     1214           
=========================================
  Files           231      231           
  Lines          5522     5522           
  Branches        266      266           
=========================================
  Hits           4617     4617           
  Misses          762      762           
  Partials        143      143           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[wslulciuc]

Good catch, @thomas-delrue. Thanks!

[boring-cyborg]

Great job! Congrats on your first merged pull request in the Marquez project!