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

Addition of CODECOV_TOKEN for eclipse-kuksa #26

Closed
erikbosch opened this issue May 13, 2024 · 4 comments
Closed

Addition of CODECOV_TOKEN for eclipse-kuksa #26

erikbosch opened this issue May 13, 2024 · 4 comments

Comments

@erikbosch
Copy link
Contributor

Kuksa-databroker uses codecov-action@v3. To use v4 we need a secret secrets.CODECOV_TOKEN. As of today we only use codecov in that particular repo, but it may in the future be used by other repos so maybe it should better be added as a orgs.newOrgSecret in this repository.

As I understand it adding it as not as easy as just creating a PR for https://github.com/eclipse-kuksa/.eclipsefdn/blob/main/otterdog/eclipse-kuksa.jsonnet as some admin activities to define/set the token is needed as well. Admins - does this issue provide sufficient information, or should we better request the change by some other mechanism, like a support ticket?

References:

FYI @SebastianSchildt @bs-jokri

@SebastianSchildt
Copy link
Contributor

here is another example ticket https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/4391

The main point is, somebody who is taking this task should also make the ticket at Eclipse help desk, because then he/she also gets notified. Especially if except the mechanical task of creating the token, there is more tech support needed afterwards. If I (or @bs-jokri ) is doing it, but we are NOT the ones integrating it here in CI, that is inconvenient communication-wise.

Anyone of you (with an Eclipse account) can make tickets in the Gitlab issue tracker of eclipsefdn. For requests like this it may be they ask, if project leads support this. In that case reach out to me or @bs-jokri and we will add to the ticket

@erikbosch
Copy link
Contributor Author

@erikbosch
Copy link
Contributor Author

@SebastianSchildt - will be fixed by #27

@erikbosch
Copy link
Contributor Author

Fixed

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

No branches or pull requests

2 participants