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

add files_dir support to allow embedding local files relative to this directory #175

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gbrackley
Copy link

This allows the use of the local contents option in a butane configuration.

e.g. include a docker-compose.yaml from a separate file:

    - path: /etc/docker-compose.yaml
      contents:
        local: docker-compose.yaml

Unless the FilesDir option is provided to the TranslateOptions then local files are not supported.

…tive to this directory

(cherry picked from commit 29fa15a)
gbrackley pushed a commit to lucidsolns/terraform-vm-proxmox that referenced this pull request Dec 17, 2023
… for butane local files

- the poseidon ct provider doesn't set the `FilesDir` option which means the whole of the butane configuration has to be in one file (unless using a remote source)
- this is likely to be switched back to 'poseidon/ct' when the pull request is merged

see:

  - poseidon/terraform-provider-ct#175
@SRv6d
Copy link

SRv6d commented Aug 27, 2024

Looks great, can someone merge this?

@freundTech
Copy link

@dghubble Any chance to get this merged and released? It has been open for almost a year with no review and is seriously limiting the usefulness of this provider.

@gbrackley
Copy link
Author

Given the stats, this forked version seems to have been found:

https://registry.terraform.io/providers/lucidsolns/ct/latest

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.

3 participants