Skip to content

[v2] Fix project service port#7059

Open
machichima wants to merge 2 commits intoflyteorg:v2from
machichima:fix-project-service-port
Open

[v2] Fix project service port#7059
machichima wants to merge 2 commits intoflyteorg:v2from
machichima:fix-project-service-port

Conversation

@machichima
Copy link
Member

@machichima machichima commented Mar 20, 2026

Tracking issue

Why are the changes needed?

When running in sandbox, we cannot link to project service as it's not set in the Envoy proxy config.
Also, we should still use port 8090 whether running with dev mode or not

What changes were proposed in this pull request?

  • Add ProjectService to envoy proxy
  • Use docker publish to 8090 rather than 30080

How was this patch tested?

Run locally with in-cluster mode, ensure we can connect to project service with

❯ ENDPOINT=http://localhost:8090 FILTERS='value_in(state,0;1;2)' bash runs/test/scripts/list_projects.sh

{
  "projects": {
    "projects": [
      {
        "id": "flytesnacks",
        "name": "flytesnacks",
        "domains": [
          {
            "id": "development",
            "name": "Development"
          },
          {
            "id": "production",
            "name": "Production"
          },
          {
            "id": "staging",
            "name": "Staging"
          }
        ]
      }
    ]
  }
}

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

@github-actions github-actions bot mentioned this pull request Mar 20, 2026
3 tasks
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.

1 participant