Deploy to AWS #34
Closed
Deploy to AWS #34
Conversation
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
amitaibu
commented
Jan 9, 2024
| @@ -12,11 +12,12 @@ JS_FILES += ${IHP}/static/vendor/turbolinksMorphdom.js | |||
|
|
|||
| include ${IHP}/Makefile.dist | |||
|
|
|||
| tailwind-dev: | |||
| tailwind-dev: node_modules | |||
There was a problem hiding this comment.
@mpscholten what is node_modules doing here?
There was a problem hiding this comment.
Oh, I guess it should be npm
so it calls make npm
and I should have
npm:
npm install
There was a problem hiding this comment.
It triggers a make node_modules when the node_modules directory doesn't exists
There was a problem hiding this comment.
so we should define also:
node_modules:
npm install
?
There was a problem hiding this comment.
It's already defined like that (npm ci is similar to bpm install)
There was a problem hiding this comment.
Thanks, moving discussion to digitallyinduced/ihp#1890
|
#!/bin/bash
# Define port
PORT=80
# Define the response
read -r -d '' RESPONSE <<EOF
HTTP/1.1 200 OK\r
Content-Length: 11\r
Content-Type: text/plain\r
\r
Hello World
EOF
# Create a TCP server on port 80
while : ; do
# Listen for a connection and respond
echo -e "$RESPONSE" | nc -l $PORT
done
Using a very simple Bash-based webserver, I tested the connectivity. Port 80 is available, but certificate validation does not succeed. |
|
|
|
Can you run |
I'd assume for now that the Nginx server is not started by the time it tries to validate the certificate. |
Perhaps that's the matter, that Nginx config refers to a certificate that does not exist yet. |
|
Hm can you run |
|
I just restarted the service, it has the same error: Almost certainly it's the failing Nginx service that's preventing ACM. |
|
Ok, let's try to disable SSL then for the moment: After that the nginx should be running |
|
|
Checking https://nixos-and-flakes.thiscute.world/nixos-with-flakes/modularize-the-configuration to be able to do the override. |
|
Success. I need to create the SQL database at this point. |
|
Hm this should happen automatically. Are you on latest IHP master? I have fixed something related to the database with digitallyinduced/ihp@ec29222 a month ago |
This reverts commit ccce5d0.
|
@mpscholten I have the same error after the update: |
|
Can you try this? https://discourse.nixos.org/t/reinstall-service-from-scratch/12514/3 |
|
@mpscholten Had the same error afterwards. But no luck... my idea was that in this file, at the 1st line, I'd create the missing database. |
This does not help either. |
|
NixOS/nixpkgs#109273 (comment) - it seems for me that |
|
@mpscholten How can I get rid of the in-server PostgreSQL server and use a managed one on AWS? |
|
I think we just need to extend the init script to also create the user + db. I just pushed a change for this. Can you switch to the new branch ( |
|
|
^^ I am going to address this in my fork. |
This is definitely much better now. |
|
@mpscholten you should have your key in this AWS env - are you able to login to it? (So we're not blocking you 😄 ) |
|
Thanks, just logged into the ec2 instance. |
|
Just saw that these env vars are set and the keys exists in the /root directory |
|
The public key is wrongly encoded. Got it working by adjusting the preStart script to this: |
This reverts commit 354f962.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.