Skip to content

Conversation

@englishm
Copy link
Collaborator

Adds support for deploying the moq-demo relay server to fly.io

@englishm
Copy link
Collaborator Author

Working now on getting the relay server demo running in GKE, but this might be useful to revisit at some point.

@englishm englishm mentioned this pull request Aug 17, 2023
Copy link
Collaborator

@kixelated kixelated left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gud Dockerfile, just needs to be updated with the latest branch.

cert/fly-secrets Outdated
@@ -0,0 +1,4 @@
#!/usr/bin/env bash

cat cert/moq-demo.englishm.net.key | base64 | flyctl secrets set --stage MOQ_KEY=-
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this file should be checked in?

entrypoint.sh Outdated
echo "$MOQ_CRT" | base64 -d > cert/moq-demo.crt
echo "$MOQ_KEY" | base64 -d > cert/moq-demo.key

# while true; do
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heh

Dockerfile Outdated

RUN mkdir -p moq-transport/src moq-transport-quinn/src moq-demo/src moq-warp/src
COPY moq-transport/Cargo.toml moq-transport/Cargo.toml
COPY moq-transport-quinn/Cargo.toml moq-transport-quinn/Cargo.toml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't exist when you update.

@englishm englishm mentioned this pull request Aug 30, 2023
@englishm englishm merged commit 48de6a3 into moq-dev:main Sep 7, 2023
ajay-s-2502 pushed a commit to ajay-s-2502/moq-rs that referenced this pull request Nov 22, 2024
Adds support for deploying the `moq-quinn` relay server to fly.io
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.

2 participants