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

Use unstable explorer client #5058

Merged
merged 11 commits into from
Oct 31, 2024
Merged

Use unstable explorer client #5058

merged 11 commits into from
Oct 31, 2024

Conversation

dynco-nym
Copy link
Contributor

@dynco-nym dynco-nym commented Oct 29, 2024

Clean up stale testruns & better logging

  • use new /unstable endpoints on explorer for backwards compatibility
  • log gw identity key
  • better agent testrun logging
  • log responses on server side
  • change response code for agents
  • update sqlx data
  • fix agent <-> probe gw bug

This change is Reviewable

@dynco-nym dynco-nym self-assigned this Oct 29, 2024
@dynco-nym dynco-nym added this to the Crunch milestone Oct 29, 2024
Copy link

vercel bot commented Oct 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-nextra ❌ Failed (Inspect) Oct 30, 2024 5:09pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
nym-explorer ⬜️ Ignored (Inspect) Visit Preview Oct 30, 2024 5:09pm
nym-next-explorer ⬜️ Ignored (Inspect) Visit Preview Oct 30, 2024 5:09pm

nym-node-status-api/Cargo.toml Outdated Show resolved Hide resolved
nym-node-status-api/src/http/api/testruns.rs Show resolved Hide resolved
nym-node-status-api/src/testruns/mod.rs Show resolved Hide resolved
@dynco-nym dynco-nym changed the base branch from develop to release/2024.13-magura October 30, 2024 13:46
@dynco-nym dynco-nym changed the base branch from release/2024.13-magura to develop October 30, 2024 14:25
@dynco-nym dynco-nym merged commit c740f84 into develop Oct 31, 2024
19 of 20 checks passed
@dynco-nym dynco-nym deleted the dz-explorer branch October 31, 2024 03:32
dynco-nym added a commit that referenced this pull request Oct 31, 2024
* Use unstable explorer client

* Clean up stale testruns & logging
- log gw identity key
- better agent testrun logging
- log responses
- change response code for agents

* Better logging on agent

* Testrun stores gw identity key instead of gw pk

* Agent 0.1.3

* Agent 0.1.4

* Sqlx offline query data + clippy

* Compatible with directory v2

* Point to internal deps + rebase + v0.1.5

* self described field not null

* Fix build.rs typo
dynco-nym added a commit that referenced this pull request Oct 31, 2024
* Use unstable explorer client

* Clean up stale testruns & logging
- log gw identity key
- better agent testrun logging
- log responses
- change response code for agents

* Better logging on agent

* Testrun stores gw identity key instead of gw pk

* Agent 0.1.3

* Agent 0.1.4

* Sqlx offline query data + clippy

* Compatible with directory v2

* Point to internal deps + rebase + v0.1.5

* self described field not null

* Fix build.rs typo
dynco-nym added a commit that referenced this pull request Nov 1, 2024
* Revert "NS API with directory v2 (#5068)"

This reverts commit cf4fe5f.

* Merge pull request #5050 from nymtech/dz-node-status-api

Node Status API

* Ns agent workflow (#5055)

* feat: add dockerfile

* add github workflow for node status agent

---------

Co-authored-by: Fran Arbanas <[email protected]>

* NS API with directory v2 (#5058)

* Use unstable explorer client

* Clean up stale testruns & logging
- log gw identity key
- better agent testrun logging
- log responses
- change response code for agents

* Better logging on agent

* Testrun stores gw identity key instead of gw pk

* Agent 0.1.3

* Agent 0.1.4

* Sqlx offline query data + clippy

* Compatible with directory v2

* Point to internal deps + rebase + v0.1.5

* self described field not null

* Fix build.rs typo

* Fix clippy

---------

Co-authored-by: Fran Arbanas <[email protected]>
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.

3 participants