Skip to content

Update k8s to enable HTTP ingress to the service #86

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

Merged
merged 2 commits into from
Nov 19, 2024
Merged

Update k8s to enable HTTP ingress to the service #86

merged 2 commits into from
Nov 19, 2024

Conversation

vcarl
Copy link
Member

@vcarl vcarl commented Nov 19, 2024

This lets us run a regular-ol website, for instance, and use Discord interaction webhooks. Interaction webhooks will unblock #59 and resolve #84

Copy link

what-the-diff bot commented Nov 19, 2024

PR Summary

  • Simplified Automation Pipeline: The system previously used Docker cache setup, which was part of GitHub Actions. This has been removed to streamline our workflow.
  • Improved Cache Strategy For Faster Docker Builds: There's been a change in how we save data during Docker image creation. Rather than storing previous versions on GitHub, we're now using a Docker registry. This means faster image builds and, consequently, quicker deployment.
  • Established Secure Connection Mechanism: We've added a 'ClusterIssuer' file, which is essentially a ticket to acquire secure certificates from a free, widely trusted provider, Let's Encrypt. This gives us the valid certificates required to improve our service's security.
  • Enhanced Website Security with SSL Redirection: Furthermore, in cluster/ingress.yaml, changes have been made to introduce SSL redirection. This upgrade automatically switches users from an insecure HTTP connection to a secure HTTPS one, increasing overall website security. Also, the secret key name was switched to letsencrypt-prod-key for stronger security consistency and structure.

@vcarl
Copy link
Member Author

vcarl commented Nov 19, 2024

Hmm

ERROR: failed to configure registry cache importer: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

We might actually be better off without it
@vcarl
Copy link
Member Author

vcarl commented Nov 19, 2024

Removed some build cache changes and swapped it to just not cache at all. That's significantly faster, previously:
Screenshot 2024-11-18 at 11 26 22 PM

and now:
Screenshot 2024-11-18 at 11 26 05 PM

@vcarl vcarl merged commit b6efec3 into main Nov 19, 2024
5 checks passed
@vcarl vcarl deleted the vc-service branch November 19, 2024 04:27
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.

Add k8s Ingress
1 participant