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

Allow specifying transport config (e.g. LXD remote) at the command line #3053

Closed
dontlaugh opened this issue Feb 9, 2022 · 3 comments
Closed
Labels
Feature New features and improvements. Stale Issues and PRs that haven't had activity in 60 days.

Comments

@dontlaugh
Copy link
Contributor

dontlaugh commented Feb 9, 2022

Use Case

I would like to target LXD remotes without using an inventory file. This is possible for some transports, like SSH, but not LXD.

Describe the Solution You Would Like

One way to do this is to change the interpretation of "host:port" syntax, if and only if --transport lxd is provided.

% bolt command run --transport lxd -t edge:proxy-1 uptime
Could not parse target URI: Invalid port number: "proxy-1"

Perhaps, in the case of LXD, this would be interpreted as remote:instance. This would rhyme with the behavior of lxc, as well, which uses the same syntax to prefix instances/containers with a remote.

% lxc exec edge:proxy-1 -- uptime
 00:11:09 up  1:28,  0 users,  load average: 0.01, 0.08, 0.04

Describe Alternatives You've Considered

If we are uncomfortable with magic parsing of bolt's target, perhaps transport configurations could be specified a different way, with a dedicated flag.

--transport-config lxd.remote=edge

or the shorter

-T lxd.remote=edge

If yall are willing to use up one of your precious short flags :)

@dontlaugh dontlaugh added the Feature New features and improvements. label Feb 9, 2022
@dontlaugh
Copy link
Contributor Author

This would impact this issue, as well martezr/packer-plugin-bolt#1

Using bolt for packer builds isn't really something that inventory is usable for. Right now SSH transport is supported, but allowing other transports would be facilitated by specifying transport configs at the command line.

@dontlaugh dontlaugh changed the title Allow specifying LXD remote at the command line Allow specifying transport config (e.g. LXD remote) at the command line Feb 28, 2022
@github-actions
Copy link

This issue has not had activity for 60 days and will be marked as stale.
If this issue continues to have no activity for 7 days, it will be closed.

@github-actions github-actions bot added the Stale Issues and PRs that haven't had activity in 60 days. label Jul 29, 2022
@github-actions
Copy link

github-actions bot commented Aug 5, 2022

This issue is stale and has been closed. If you believe this is in error,
or would like the Bolt team to reconsider it, please reopen the issue.

@github-actions github-actions bot closed this as completed Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New features and improvements. Stale Issues and PRs that haven't had activity in 60 days.
Projects
None yet
Development

No branches or pull requests

1 participant