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

sync code with upstream #10

Merged
merged 78 commits into from
May 13, 2020
Merged

sync code with upstream #10

merged 78 commits into from
May 13, 2020

Conversation

shabir61
Copy link

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

nithu0115 and others added 30 commits June 25, 2019 09:17
* Moving log collector script to this repo

* Added changes according to 1.4.1
The instructions on readme were still pointing at the original repository. Updating to reflect the new location
* added a set of variables to allow private ssh to non-default vpc

* make filepaths of ./files/ and install-worker relative to packer template dir

* updated ami_description to a variable
* Add all new instance types already added to the CNI

* Add support for the u-*tb1.metal instances (Fix awslabs#319)
* sync eks node group template to be latest available
1. add support to use ssm parameter for amiID
2. add support for all instance types supported by cni
3. formatted with rain(https://github.com/aws-cloudformation/rain)

* add new CFN version 2019-09-17
Add 1.14 to the list of Makefile targets.

Remove 1.10 as it's no longer a supported version

Update versions and build dates for older EKS versions
This name of the AutoScaling Group is useful for things like the Cluster Autoscaler so that it can manage automatic cluster scaling.
Claes Mogren and others added 19 commits March 25, 2020 13:14
"the -f is force, -o is overwrite, meaning if there is an existing group with number 1950, it will create a new one with the name docker"
…ded NodeSecurityGroup dependency to SG Ingress/Egress (awslabs#450)

Co-authored-by: Vishal Gupta <[email protected]>
…ithub. (awslabs#457)

The default behavior is unchanged and will still pull assets from
Github.
* updated ipamd data file extension to json

* updated ipamd metrics file extension
This commit adds a new manifest which contains AMI name in the manifest filename so that parallel builds can be triggered. Even though the new manifest is now generated along with the current one for backwards compatibility, eventually the old manifest (manifest.json) will be deprecated.
scripts/install-worker.sh Outdated Show resolved Hide resolved
@@ -7,6 +7,6 @@ set -o errexit
#sudo yum update -y kernel
#sudo amazon-linux-extras install kernel-ng
sudo yum -y install kernel-4.14.133-113.112.amzn2.x86_64
sudo grubby --set-default /boot/vmlinuz-4.14.133-113.112.amzn2.x86_64
sudo grubby --set-default /boot/vmlinuz-4.14.133-113.112.amzn2.x86_64 --args="ro console=tty0 console=ttyS0,115200n8 net.ifnames=0 biosdevname=0 nvme_core.io_timeout=4294967295 rd.emergency=poweroff rd.shell=0 LANG=en_US.UTF-7 KEYTABLE=us udev.event-timeout=300"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets undo installing the specific kernel, lte it install the latest version available by default since we now know that that was not the issue;

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure how i pass this value then :-). udev.event-timeout=300

Copy link

@sakomws sakomws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@sakomws sakomws merged commit 19ad5d6 into master May 13, 2020
@sakomws sakomws deleted the bugger branch May 13, 2020 02:29
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.