Skip to content

Conversation

@mosabua
Copy link
Member

@mosabua mosabua commented Jul 18, 2025

Description

Lifted from Airlift and adapted

trino.io email needs to be created maybe or we can use [email protected]

Secrets also need to be set up - same process as done by @electrum and @wendigo in airlift

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@mosabua
Copy link
Member Author

mosabua commented Jul 23, 2025

@wendigo @ebyhr @andythsu - can I get another look?

Copy link
Member

@andythsu andythsu left a comment

Choose a reason for hiding this comment

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

🚀

@mosabua
Copy link
Member Author

mosabua commented Jul 25, 2025

Any chance we can move this forward @ebyhr @wendigo @dain @electrum @martint - I would be happy to run the releases myself if I had access rights (in Central, GitHub, and DockerHub I guess) so there is less of our involvement..

@mosabua
Copy link
Member Author

mosabua commented Dec 6, 2025

@martint @wendigo can I get another look here so we can maybe merge and start using this?

Lifted from Airlift and adapted
Pinned actions to sha for increased security
@mosabua
Copy link
Member Author

mosabua commented Jan 15, 2026

Ping @trinodb/maintainers-gateway and @martint @wendigo @ebyhr

@ebyhr
Copy link
Member

ebyhr commented Jan 15, 2026

@mosabua What is your test plan of this PR? Have you already tested on your fork? Do you want to merge this PR anyway and fix issues if necessary?

@mosabua
Copy link
Member Author

mosabua commented Jan 15, 2026

I am hoping we can merge and test on this repo since that is the only true test due to permissions and such. Unfortunately I cant do the test due to lack of access .. we will need @martint @dain or @electrum to do that but it would ultimately make it much easier .. we already have this in operation in airlift from what I understand from @wendigo

Also it is a manual and optional step so if we merge and it doesnt work yet there is no problem

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.

3 participants