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

[BUG] Setting the build_password using set-envvars.sh #197

Closed
1 task done
evilaliv3 opened this issue Apr 13, 2022 · 3 comments · Fixed by #198
Closed
1 task done

[BUG] Setting the build_password using set-envvars.sh #197

evilaliv3 opened this issue Apr 13, 2022 · 3 comments · Fixed by #198
Assignees
Labels

Comments

@evilaliv3
Copy link
Contributor

evilaliv3 commented Apr 13, 2022

Code of Conduct

  • I have read and agree to the project's Code of Conduct.

Project Version

main

VMware vSphere

7.0 Update 2

HashiCorp Packer

v1.8.0

HashiCorp Packer Plugin for VMware vSphere

v1.0.3

Guest Operating System

Any Supported

Environment Details

The current packer templates seem to not use the build_password variable so that the only password read by packer is the one that the user provides in the install script

I've noticed this situation while working on creating a Debian template starting from the existing template for Ubuntu 18.04.

In order to fix this issue and use the build_password variable provided within the set-envvars.sh script the following instruction should be added to the templates:

build_password = var.build_password

Description

Expected Behavior

Actual Behavior

Steps to Reproduce

Run a build setting a password buid_password in the config file as "password"
Run the set-envvars.sh script and set the password to "different_password"
Run the build.

Result: the password used by packer will be "password" and not "different_password" as expected.

Log Fragments and Files

No response

Screenshots

No response

Additional Context

@tenthirtyam tenthirtyam changed the title [BUG] Impossibility to set build_password by using the set-envvars.sh script [BUG] Setting the build_password using set-envvars.sh Apr 13, 2022
@tenthirtyam tenthirtyam added priority/medium Priority Medium sev/low Low size/s Small labels Apr 13, 2022
@evilaliv3
Copy link
Contributor Author

Looking at the code i think this change shall be applied to alle the Linux distributions.

I'm instead not sure what to do for the rest of the operating systems.

@tenthirtyam
Copy link
Collaborator

tenthirtyam commented Apr 13, 2022

I'll take a look at it soon and address it. I updated the formatting and data in the originating bug content.

Ryan Johnson
Staff II Solutions Architect | VMware, Inc.

evilaliv3 added a commit to evilaliv3/packer-examples-for-vsphere that referenced this issue Apr 13, 2022
@evilaliv3 evilaliv3 mentioned this issue Apr 13, 2022
10 tasks
evilaliv3 added a commit to evilaliv3/packer-examples-for-vsphere that referenced this issue Apr 13, 2022
evilaliv3 added a commit to evilaliv3/packer-examples-for-vsphere that referenced this issue Apr 13, 2022
tenthirtyam pushed a commit to evilaliv3/packer-examples-for-vsphere that referenced this issue Apr 13, 2022
tenthirtyam pushed a commit that referenced this issue Apr 13, 2022
* Fixes GH-197.
* Update `CHANGELOG.md`

Co-authored-by: Ryan Johnson <johnsonryan@vmware.com>
@github-actions
Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 14, 2022
charkssmith pushed a commit to charkssmith/packer-examples-for-vsphere that referenced this issue Aug 28, 2023
* Fixes vmware-samplesGH-197.
* Update `CHANGELOG.md`

Co-authored-by: Ryan Johnson <johnsonryan@vmware.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants