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

Reboot after updating packages in packer. #90

Closed
wants to merge 1 commit into from
Closed

Reboot after updating packages in packer. #90

wants to merge 1 commit into from

Conversation

rajachan
Copy link
Member

@rajachan rajachan commented Jan 9, 2018

_update_pacakges would almost always bring in an updated kernel. Invoking the
cfncluster::default recipe without rebooting the instance right after causes
issues with recipes like the ones that install NVIDIA drivers, which pulls down
a version of kernel-devel newer than what the instance is running with at that
time.

The pause_before timeout is a bit conservative but gives plenty of time for
packer to reconnect to the instance after the reboot and before trying to run
the following provisioner.

Signed-off-by: Raghu Raja [email protected]

As part of 5f3ed72 that added the _update_packages recipe to packer's
run_list, the APT update was removed to avoid redundancy. We install
build-essentials and wget to pull down and install Chef, which in turn
runs _update_packages.  Not doing an apt-get update prior to that ends
up looking for non-existent libc and other libraries, so adding back the
redundant update.

Signed-off-by: Raghu Raja <[email protected]>
@rajachan rajachan closed this Jan 9, 2018
@rajachan rajachan deleted the nvidia-kerndev branch January 9, 2018 00:58
sean-smith added a commit to sean-smith/cfncluster-cookbook that referenced this pull request Feb 22, 2019
Fixes issue with travis caused by [aws#80](dtao/safe_yaml#80) and fixed in [aws#90](dtao/safe_yaml#90)

To test:

	bundle install
	bundle exec rake

Signed-off-by: Sean Smith <[email protected]>
sean-smith added a commit that referenced this pull request Feb 22, 2019
Fixes issue with travis caused by [#80](dtao/safe_yaml#80) and fixed in [#90](dtao/safe_yaml#90)

To test:

	bundle install
	bundle exec rake

Signed-off-by: Sean Smith <[email protected]>
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.

1 participant