hashicorp / hashicorp/packer-plugin-azure
azure: Communication configuration
- Dominant language
- Go
- Stars
- 63
- Forks
- 103
- PR merge metrics
- No merged PRs in 30d
Description
_This issue was originally opened by @rickard-von-essen in https://github.com/hashicorp/packer/issues/4174 and has been migrated to this repository. The original issue description is below._
---
Related to #4172
- [ ] Don't set a default `ssh_username` [in a builder](https://github.com/hashicorp/packer/blob/676c3a21bf1858d5786ea3c74efbd1e7a2a24ad4/builder/azure/arm/config.go#L688).
- [ ] [The default to both](https://github.com/hashicorp/packer/blob/676c3a21bf1858d5786ea3c74efbd1e7a2a24ad4/builder/azure/arm/config.go#L580) `ssh`and `winrm` communicator is really weird and undocumented. I think it should be removed, even if it breaks BC, or at least clearly documented. Currently some valid configurations will be ignored or not work as expected.
Contributor guide
Research direction
Start in builder/azure/arm/config.go at the referenced lines around the ssh_username and communicator defaults, and review related issue #4172. Determine how the defaults affect valid SSH and WinRM configurations, then make the behavior explicit or documented and verify that the listed configuration cases are handled as intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, go
- Domain
- cloud
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100