-
Notifications
You must be signed in to change notification settings - Fork 266
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
chore: simplify otel config, 1val setup, fix pod dns, retries #10344
Merged
Merged
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
b418f24
show ethereum node logs
ludamad 19bf367
fix
ludamad be93a1e
remove gotcha from deploy spartan script
ludamad e55f705
fix
ludamad 19923b2
try to fix OTEL_COLLECTOR_ENDPOINT
ludamad b1f5fcd
try to fix OTEL_COLLECTOR_ENDPOINT
ludamad 432d9b8
try to fix OTEL_COLLECTOR_ENDPOINT
ludamad 978a0b6
perms
ludamad 7e2ec05
validator set
ludamad 556590a
fix
ludamad 0449974
retry on l1 contract depoy
ludamad 1538531
verifier deploy fix
ludamad 2059d4b
verifier deploy fix
ludamad c8684e4
fix
ludamad e0263a9
retries
ludamad e7bf831
values file
ludamad 550ce53
one more try...
ludamad 6be0148
paranoia
ludamad 576ac0f
try with service disabled?
ludamad 2367cbf
-
ludamad 398ccee
fix broker being external
ludamad d8044f4
dns policy
ludamad b88c328
fixes
ludamad d1fb96e
helper
ludamad 1ec3679
detect telemetry and dont set endpoint
ludamad f0b774b
devnet fix
ludamad 2d4dcd4
bot settings
ludamad 1bd2ba1
Merge branch 'master' into ad/fix/deploy_spartan.sh
ludamad 0614f4a
chore: devnet extras (#10414)
alexghr e3ddd72
Fixes
PhilWindle 54fc32e
Merge remote-tracking branch 'origin/master' into ad/fix/deploy_spart…
PhilWindle e2d6be4
Remove collector endpoint config
PhilWindle 292e351
Merge remote-tracking branch 'origin/master' into ad/fix/deploy_spart…
PhilWindle 8a8d600
Fix
PhilWindle 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
detect telemetry and dont set endpoint
- Loading branch information
commit 1ec367923a34c460f9bf349f9dd6810dd8ab0772
There are no files selected for viewing
This file contains 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 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 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 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otel collector port is not provided here although it is available for use in the script above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we want to keep it as the default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I wanted to make it available potentially in future but it is a bit misleading. You can change the port by changing the full URL but no real reason to change ports imo