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

caa: use our own image #1035

Closed
wants to merge 1 commit into from
Closed

caa: use our own image #1035

wants to merge 1 commit into from

Conversation

burgerdev
Copy link
Contributor

Adjust the upstream cloud-api-adaptor DaemonSet to the idiosyncracies of our Nix-built container image, and vice-versa.

  • Set the HOME env var so that the app can find the SSH key.
  • Add a mount point for /var/run/netns because our image does not symlink /var/run to /run.
  • Add a stable symlink to the real entrypoint at the expected location.

Furthermore, add the required iptables binary as a runtime dependency to cloud-api-adaptor.

Adjust the upstream cloud-api-adaptor DaemonSet to the idiosyncracies of
our Nix-built container image, and vice-versa.

* Set the HOME env var so that the app can find the SSH key.
* Add a mount point for /var/run/netns because our image does not
  symlink /var/run to /run.
* Add a stable symlink to the real entrypoint at the expected location.

Furthermore, add the required iptables binary as a runtime dependency to
cloud-api-adaptor.
@burgerdev burgerdev added the no changelog PRs not listed in the release notes label Nov 27, 2024
@burgerdev burgerdev marked this pull request as draft November 27, 2024 13:32
@burgerdev
Copy link
Contributor Author

Sorry, I'm gonna withdraw this for now because I discovered that it does not work well with the reverted node-installer changes.

@burgerdev burgerdev closed this Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant