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

k8s docker-compose #154

Merged
merged 15 commits into from
Oct 25, 2022
Merged

Conversation

jacobweinstock
Copy link
Member

@jacobweinstock jacobweinstock commented Oct 3, 2022

Description

This PR brings up the sandbox via Docker compose using the Kubernetes backend for all service. This does not completely remove the postgres backend setup but moves all the compose with postgres into an isolated directory (deploy/compose/postgres) that can be removed when we're ready.

I did not touch the terraform setup. I need some help validating that one. please and thank you. CC @mmlb @displague

Why is this needed

Fixes: #142 #45 #118 #131 #133 #145 #148

  • This "fixes" a quite a few issues related to TLS cert generation. This is the case because we are not using TLS in this deployment. Also see, Intent to drop tls as default tink#555.
  • This also "fixes" any issues related to the internal registry as that is removed as the default.

How Has This Been Tested?

Manually tested vagrant with virtualbox (on a Mac), vagrant with libvirt (on Ubuntu 22.04), and docker-compose (on on Ubuntu 22.04).

How are existing users impacted? What migration steps/scripts do we need?

There is no migration support. Users will need to follow a quick start guide to get started.

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

This compose runs all services using their kubernetes
backend and run a K3S cluster as the datastore.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
`BACKEND_KUBE=true vagrant up`

This will enable the kubernetes based tink stack in vagrant.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
This is needed in source control.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
This handles customizing the disk type when using libvirt
vs virtualbox.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Couldn't find a reliable way to set the disk based on
the provider in the Vagrantfile.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
The output was not accurate before. Now it is.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
deploy/vagrant/setup.sh Outdated Show resolved Hide resolved
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock force-pushed the k8s-docker-compose branch 3 times, most recently from 5c1817c to 355e42a Compare October 3, 2022 19:04
With the official deprecation of the postgres backend, this
deploys the stack using the kubernetes backend in docker-compose.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
This gets Rufio up and running the in the stack.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock added kind/feature Categorizes issue or PR as related to a new feature. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Oct 3, 2022
@jacobweinstock jacobweinstock marked this pull request as ready for review October 3, 2022 20:02
@mmlb
Copy link
Contributor

mmlb commented Oct 4, 2022

looks pretty good to me, if all looks good after I run it we can merge

This allows us to use the newer version syntax.
And version 1.29.0 is quite old and was in need updating.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. and removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Oct 11, 2022
Previously we were unable to use tag versions for
tink-controller and tink-worker. That is no longer the
case.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Oct 25, 2022
@jacobweinstock jacobweinstock removed the request for review from mmlb October 25, 2022 19:47
@mergify mergify bot merged commit f180972 into tinkerbell:main Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment