-
Notifications
You must be signed in to change notification settings - Fork 11
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
Rust-based stats server on fly.io #165
Conversation
ee4e74c
to
7a5d7f5
Compare
This reverts commit 895f413.
Co-authored-by: Jiahao XU <[email protected]> Signed-off-by: David Laban <[email protected]>
Co-authored-by: Jiahao XU <[email protected]> Signed-off-by: David Laban <[email protected]>
server/fly.io/Cargo.toml
Outdated
tokio = { version = "1", features = ["rt-multi-thread", "macros"] } | ||
axum = "0.6.10" | ||
influx_db_client = "^0.5.0" | ||
influxrs = "2.0.1" |
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.
placeholder to think about ijagberg/influx#4 at some point. This might involve forking their repo temporarily and deploying from the fork
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.
oh you know what? it might be a problem with how the js server is encoding the url params?
(still not looked into it properly though)
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.
Fixed in ijagberg/influx#5 . Depending on my fork for now.
edition = "2021" | ||
|
||
# Make a new workspace so that we get our own Cargo.lock and target dir for the docker build. | ||
[workspace] |
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.
For some reason my vscode is no longer giving any hints for anything in this folder unless I explicitly open only this folder in its own instance. I'm sure it worked before, and for other projects, but I can't remember what I did last time.
Probably I will merge this as is, but if this causes problems for anyone else then tell me and I'll try to address it properly.
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.
Roughly following https://fly.io/docs/launch/continuous-deployment-with-github-actions/ but using an environment to hold the secret, so you can only access the fly secret if you can push to main.
This was tested on a previous commit.
Logs exist here: https://github.com/cargo-bins/cargo-quickinstall/actions/runs/10761262315/job/29840247483?pr=165
Co-authored-by: Jiahao XU <[email protected]> Signed-off-by: David Laban <[email protected]>
working towards #164
To test what I have done so far (forwarding from vercel to fly.io and logging query params):
cargo quickinstall --force cargo-quickinstall
and see what gets printed in the fly.io logs aboveI have made a start on a public dashboard at https://alsuren.grafana.net/public-dashboards/12d4ec3edf2548a1850a813e00592b53 . I think I might make a public cargo-bins org with a shared login or something, so we can play about with it more freely.
TODO:
/record-install?crate=$crate&version=$version&arch=$arch&client=$client