Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update config path path for Trino Gateway and release chart #310

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

oneonestar
Copy link
Member

Merge after upgrade to Trino Gateway 15

Update config path path for gateway.

Changes:

  • /etc/gateway/config.yaml to /etc/trino-gateway/config.yaml
  • /usr/lib/trino -> /usr/lib/trino-gateway

Ref:

@mosabua
Copy link
Member

mosabua commented Mar 12, 2025

I pushed the app and chart version bump in here so we can get a green build @oneonestar and @nineinchnick

@mosabua
Copy link
Member

mosabua commented Mar 12, 2025

Closing the other PR

@mosabua mosabua changed the title Update config path path for gateway Update config path path for Trino Gateway and release chart Mar 12, 2025
@mosabua mosabua force-pushed the star/gateway-config branch from 98cebb2 to aab6e33 Compare March 13, 2025 00:02
@oneonestar oneonestar force-pushed the star/gateway-config branch from aab6e33 to d3a276e Compare March 13, 2025 00:35
@oneonestar oneonestar force-pushed the star/gateway-config branch 2 times, most recently from 5fbb945 to c5b5b79 Compare March 13, 2025 01:02
@oneonestar oneonestar force-pushed the star/gateway-config branch from c5b5b79 to f54afb1 Compare March 13, 2025 01:11
@mosabua mosabua self-requested a review March 13, 2025 01:41
@mosabua
Copy link
Member

mosabua commented Mar 13, 2025

Build is green .. shipping!

@mosabua mosabua merged commit 0dcf72e into trinodb:main Mar 13, 2025
10 checks passed
Copy link
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

All good! Thx @oneonestar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants