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

Replace osrf-migration token by osrfbuild-token in bottle_job_builder #1211

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

j-rivero
Copy link
Contributor

While removing the osrf-migration token I noticed that it is being used in bottle_job_builder for authenticating the hub cli . We need it even if there are no push/clone operations involved, otherwise the hub cli is asking for credentials.

This PR changes the osrf-migration user to use existing osrfbuild token.

@scpeters
Copy link
Contributor

I'll find an opportunity to test this

@scpeters
Copy link
Contributor

build bottle

@scpeters
Copy link
Contributor

build bottle

lol, wrong PR

@scpeters
Copy link
Contributor

scpeters commented Nov 20, 2024

I'll find an opportunity to test this

I deployed the DSL from this branch:

I also manually configured the triggered builder job to use this release-tools branch and started a bottle build for osrf/homebrew-simulation#2890

@scpeters
Copy link
Contributor

it works! I'll revert the configuration and approve

@scpeters
Copy link
Contributor

it works! I'll revert the configuration and approve

https://build.osrfoundation.org/job/_dsl_brew_release/935/

@j-rivero
Copy link
Contributor Author

Thanks for the testing!

@j-rivero j-rivero merged commit 96bcff2 into master Nov 21, 2024
1 check passed
@j-rivero j-rivero deleted the jrivero/remove_migration branch November 21, 2024 10:00
@j-rivero j-rivero restored the jrivero/remove_migration branch November 21, 2024 10:00
@j-rivero j-rivero deleted the jrivero/remove_migration branch November 21, 2024 10:00
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.

2 participants