Skip to content

Conversation

@marun
Copy link

@marun marun commented Aug 21, 2024

Why this should be merged

The new monitoring stack is located at different URLs.

How this works

How this was tested

  • Load and warp jobs are emitting the new URLs
  • New URLs are populated with data.

How is this documented

  • N/A

@marun marun added ci testing This primarly focuses on testing labels Aug 21, 2024
@marun marun self-assigned this Aug 21, 2024
@marun marun requested review from ceyonur and darioush as code owners August 21, 2024 20:47
darioush
darioush previously approved these changes Aug 21, 2024
@darioush darioush enabled auto-merge (squash) August 21, 2024 20:56
@marun marun marked this pull request as draft August 21, 2024 20:58
auto-merge was automatically disabled August 21, 2024 20:58

Pull request was converted to draft

@marun
Copy link
Author

marun commented Aug 21, 2024

Converted to draft to avoid merging prematurely. I realized I need to have a tag for 35c66e33f in avalanchego and then update the action usage in this PR to use the new tag.

@marun marun marked this pull request as ready for review August 21, 2024 21:12
@marun
Copy link
Author

marun commented Aug 21, 2024

Updated to use the current action, and manually checked that the warp and load test jobs were emitting correct links and sending metrics and logs.

@darioush darioush merged commit 32c8cdf into master Aug 22, 2024
@darioush darioush deleted the tmpnet-update-monitoring-urls branch August 22, 2024 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci testing This primarly focuses on testing

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants