Skip to content

Conversation

@yacovm
Copy link
Contributor

@yacovm yacovm commented Mar 6, 2025

Why this should be merged

In order to better know the exact code version that is run, it's useful if the node would print it upon startup.

How this works

Just prints the commit which is set at compile time.

How this was tested

built the binary and ran it.

Need to be documented in RELEASES.md?

no

maru-ava and others added 2 commits March 10, 2025 19:14
Jobs executing on PRs based on fork branches won't have access to the
secrets required to configure collectors for logs and
metrics. Disabling the collection in the custom action avoids failing
such jobs. This was the behavior prior to the switch to starting
collectors with golang.
Signed-off-by: Yacov Manevich <yacov.manevich@avalabs.org>
@StephenButtolph StephenButtolph added this pull request to the merge queue Mar 10, 2025
Merged via the queue into ava-labs:master with commit fbfc9aa Mar 10, 2025
23 checks passed
cam-schultz pushed a commit that referenced this pull request Mar 24, 2025
Signed-off-by: Yacov Manevich <yacov.manevich@avalabs.org>
Co-authored-by: Maru Newby <maru.newby@avalabs.org>
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.

5 participants