forked from google/trillian
-
Notifications
You must be signed in to change notification settings - Fork 1
RA-8279: feat(otel): Add OpenTelemetry distributed tracing and ctutils logging integration #3
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
Open
himaschal
wants to merge
58
commits into
master
Choose a base branch
from
RA-8279_improve_application_visibility_timing_logs
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
952a94b
feat(otel): Add OpenTelemetry distributed tracing and ctutils logging…
himaschal 518b1d8
Update ctutils to v0.1.7
himaschal c64d10e
ci: add workflow to update ctutils dependencies
himaschal 86dc36a
fix(ci): use oauth2 syntax for authentication and debug token
himaschal fea9c9d
fix(ci): use GITHUB_TOKEN for PR creation
himaschal c370095
fix(ci): auth to private ctutils for lint and vuln checks
himaschal 0991a95
fix(ci): auth to private ctutils for database integration tests
himaschal 03dc543
fix(ci): auth to private ctutils and set GOPRIVATE for CodeQL
himaschal 13a414e
test: clean run trigger for v0.1.11-test
himaschal c71aacb
chore(deps): update ctutils to latest version
himaschal 6ae67f5
Merge pull request #4 from digicert/chore/update-ctutils
himaschal febb1a9
chore: test pipeline run with ctutils v0.1.12-test
himaschal 683f18f
chore(deps): update ctutils to latest version
himaschal f04e183
Merge pull request #5 from digicert/chore/update-ctutils
himaschal 66422c4
chore: test pipeline run with ctutils v0.1.13-test
himaschal 53ae555
chore(deps): update ctutils to latest version
himaschal c2fab3b
Merge pull request #6 from digicert/chore/update-ctutils
himaschal d81bf44
chore: revert pipeline testing configurations to production ready state
himaschal 996c9dc
RA-8279: Modified to use github app
himaschal ac7a3ca
ci: use github app for private repo access
himaschal 8448274
ci: update all workflows to use github app token
himaschal 4300dd0
chore(deps): update ctutils to latest version
himaschal 80f9e8f
Merge pull request #8 from digicert/chore/update-ctutils-test
himaschal 221044d
chore(deps): update ctutils to latest version
himaschal ef8cfbb
Merge pull request #9 from digicert/chore/update-ctutils-test
himaschal c16b7e8
chore(deps): update ctutils to latest version
himaschal 6704de0
Merge pull request #10 from digicert/chore/update-ctutils-test
himaschal 81f2939
fix(ci): remove hardcoded base branch reference
himaschal 07177d7
chore: update ctutils to v0.1.18-test
himaschal d64493d
Revert "chore: update ctutils to v0.1.18-test"
himaschal 370d288
chore: update ctutils to v0.1.19-test
himaschal ef8b47c
Merge pull request #12 from digicert/clean-run-6.0-update
himaschal c386558
Address Trillian PR comments in serverutil
himaschal 318ff24
Document InitLogging fail-safe behavior
himaschal 0c53a44
Document InitLogging behavior in log_signer
himaschal 74e42c3
Support LOG_LEVEL env var in config
himaschal c9543cf
Add unit tests for InitLogging
himaschal 6050b10
Remove obsolete Dockerfile.unified
himaschal cb3ffff
Clean up duplicate comments in Dockerfiles
himaschal 586f41b
Remove misleading comment in go.mod
himaschal deb3c87
Fix lint errors in config_test.go
himaschal 0e27de6
Add shutdown handling and config consistency
himaschal ff55056
add Dockerfile.local to .gitignore. Used for local dev / builds.
himaschal 2c11080
build: update ctutils dependency to fix CI build
himaschal fef833f
RA-8279: Update README with minor changes.
himaschal ba2ccae
RA-8279: Address PR comments. Add `GOPRIVATE` and `GONOSUMDB` environ…
himaschal beb4e3f
fix(ci): robust latest release resolution for ctutils
himaschal 800d4ea
fix(ci): add GONOPROXY for private ctutils module access
himaschal 4236c87
test: remove GONOPROXY to verify if it's necessary
himaschal abf3be0
Revert "test: remove GONOPROXY to verify if it's necessary"
himaschal 646baff
test: remove GONOPROXY again to check if fixed tag resolves issue
himaschal abaae85
Revert "test: remove GONOPROXY again to check if fixed tag resolves i…
himaschal abe8d6d
chore(deps): update ctutils to v0.1.27-test (#17)
github-actions[bot] 1ddf0a9
chore: update GONOPROXY indentation in workflow files and add local D…
himaschal d1e92cb
RA-8179: improve Dockerfiles for local development by adding GOPROXY …
himaschal 7fdd44d
chore(deps): update ctutils to v0.0.1-golem (#18)
github-actions[bot] 7b49d54
chore(deps): update ctutils to v0.0.2-golem (#19)
himaschal bfbfe7a
RA-8279: Do not trace admin endpoints on the backend. These are unnec…
himaschal File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,95 @@ | ||
| name: Update ctutils dependencies | ||
|
|
||
| on: | ||
|
|
||
| workflow_dispatch: | ||
| inputs: | ||
| version: | ||
| description: "Version to install (e.g. v0.1.9-test). Defaults to latest." | ||
| required: false | ||
| default: "latest" | ||
| schedule: | ||
| - cron: "0 5 * * *" # Runs daily at 05:00 UTC | ||
|
|
||
| jobs: | ||
| update-ctutils: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup Go | ||
| uses: actions/setup-go@v5 | ||
| with: | ||
| go-version-file: go.mod | ||
|
|
||
| - name: Generate token | ||
| id: generate_token | ||
| uses: actions/create-github-app-token@v1 | ||
| with: | ||
| app-id: ${{ secrets.CT_APP_ID }} | ||
| private-key: ${{ secrets.CT_APP_PRIVATE_KEY }} | ||
| owner: digicert | ||
| repositories: ctutils | ||
|
|
||
| - name: Update ctutils | ||
| env: | ||
| GOPRIVATE: github.com/digicert/ctutils | ||
| GONOSUMDB: github.com/digicert/ctutils | ||
| GONOPROXY: github.com/digicert/ctutils | ||
| GITHUB_APP_TOKEN: ${{ steps.generate_token.outputs.token }} | ||
| VERSION_INPUT: ${{ github.event.inputs.version || 'latest' }} | ||
| run: | | ||
| set -euo pipefail | ||
|
|
||
| # Switch to x-access-token syntax which is standard for App tokens | ||
| git config --global url."https://x-access-token:$GITHUB_APP_TOKEN@github.com/".insteadOf "https://github.com/" | ||
|
|
||
| RESOLVED_VERSION="$VERSION_INPUT" | ||
| if [ "$VERSION_INPUT" = "latest" ]; then | ||
| echo "Resolving latest ctutils release from GitHub API..." | ||
| # Try getting the official 'latest' release first | ||
| API_RESPONSE=$(curl -sS -H "Authorization: Bearer $GITHUB_APP_TOKEN" \ | ||
| -H "Accept: application/vnd.github+json" \ | ||
| https://api.github.com/repos/digicert/ctutils/releases/latest || true) | ||
|
|
||
| RESOLVED_VERSION=$(printf '%s' "$API_RESPONSE" | jq -r '.tag_name // empty') | ||
|
|
||
| if [ -z "$RESOLVED_VERSION" ] || [ "$RESOLVED_VERSION" = "null" ]; then | ||
| echo "Official 'latest' release not found (possibly all pre-releases). Falling back to most recent release..." | ||
| API_RESPONSE=$(curl -sS -H "Authorization: Bearer $GITHUB_APP_TOKEN" \ | ||
| -H "Accept: application/vnd.github+json" \ | ||
| "https://api.github.com/repos/digicert/ctutils/releases?per_page=1") | ||
| RESOLVED_VERSION=$(printf '%s' "$API_RESPONSE" | jq -r '.[0].tag_name // empty') | ||
| fi | ||
|
|
||
| if [ -z "$RESOLVED_VERSION" ] || [ "$RESOLVED_VERSION" = "null" ]; then | ||
| echo "Error: Unable to resolve any ctutils release tag from GitHub API." | ||
| echo "API response was: $API_RESPONSE" | ||
| exit 1 | ||
| fi | ||
| echo "Resolved version: $RESOLVED_VERSION" | ||
| fi | ||
|
|
||
| echo "VERSION_USED=$RESOLVED_VERSION" >> "$GITHUB_ENV" | ||
| echo "Updating ctutils to $RESOLVED_VERSION..." | ||
| go get github.com/digicert/ctutils@$RESOLVED_VERSION | ||
| go mod tidy | ||
|
|
||
| - name: Create Pull Request | ||
| uses: peter-evans/create-pull-request@v7 | ||
| with: | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| commit-message: "chore(deps): update ctutils to ${{ env.VERSION_USED }}" | ||
| committer: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> | ||
| author: ${{ github.actor }} <${{ github.actor }}@users.noreply.github.com> | ||
| branch: chore/update-ctutils-test | ||
|
|
||
| delete-branch: true | ||
| title: "chore(deps): update ctutils to ${{ env.VERSION_USED }}" | ||
| body: | | ||
| Updates [ctutils](https://github.com/digicert/ctutils) to version `${{ env.VERSION_USED }}`. | ||
|
|
||
| Auto-generated by GitHub Actions. | ||
| labels: | | ||
| dependencies | ||
| automated pr |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.