Commit Graph

59 Commits

Author SHA1 Message Date
justinsb 6e036b39c2 refactor: move more nodeup script logic to NodeUpScript
This enables simpler reuse.
2024-08-30 09:14:49 -04:00
justinsb 192b880434 chore: refactor MirroredAsset into assets
This seems logically coherent, and is part of the work to start using
compiled-in hashes for most well-known assets.
2024-03-30 10:34:27 -04:00
Ciprian Hacman 3325f260f8 Fix shellcheck warnings for bootstrap script 2024-03-03 13:29:31 +02:00
John Gardiner Myers 688097bd85 Remove unnecessary data from userdata 2023-08-09 18:12:37 -07:00
justinsb 6123c9e7ba nodeup script: accept strongly typed nodeup.Config
More self-documenting and flexible than accepting a string.
2022-10-08 08:50:42 -04:00
srikiz f837ced218 Fix journald for do
Incorporate review comments

Incorporate review comments

Remove echo
2022-06-07 14:25:23 +05:30
Ole Markus With ce2e877aeb Remove bazel files from vendor 2022-04-12 13:29:03 +02:00
justinsb fdf412cdf5 nodeup bash script: use explicit return code
In bash, `return` returns the exit code of the last statement.  Being
explicit here is safer, and I believe in the first case (the cached
download) the value was actually incorrect.
2021-12-20 10:08:06 -05:00
justinsb e2c28b062b Refactor nodeup script to avoid action-at-a-distance 2021-12-17 18:33:44 -05:00
justinsb 0e1bbd58bd Avoid double-encoding templates
We were previously passing the template (with placeholders) through
MIME encoding.  This seems like it might fail rarely in
hard-to-understand ways.
2021-12-17 16:28:09 -05:00
Ciprian Hacman ea7df00719 Run hack/update-gofmt.sh 2021-12-01 22:39:50 +02:00
John Gardiner Myers 74a44c2270 Don't restrict nodeup download to IPv4 2021-06-13 21:46:58 -07:00
John Gardiner Myers b45c0b4489 Remove InstanceGroup from NodeupModelContext 2021-06-03 21:27:01 -07:00
Ciprian Hacman 3b80de3bcc Convert all indents to spaces in node bootstrap script 2021-05-27 11:21:52 +03:00
John Gardiner Myers 602d1be0ba Don't download nodeup if already in the AMI 2021-05-19 07:57:16 -07:00
John Gardiner Myers e2b84efd11 Remove dead code in bootstrap script 2021-05-18 21:46:31 -07:00
Justin SB 1d76a15f69 Set the tcp_rmem sysctl in bootstrap script
This ensures that we're using our settings for downloading nodeup
itself and any assets that nodeup downloads.  This is a workaround for
reported problems with the initial download on some kernels otherwise.

Issue #10206
2021-01-24 21:50:45 -05:00
Rodrigo Menezes 3fb12c66ae gzip and base64 encode the heredocs in the nodeup.sh portion of user-data 2020-12-04 10:46:18 -08:00
Rodrigo Menezes c9af4de9cf Remove copywrite from nodeup scripts to reduce the user-data size 2020-11-30 12:49:25 -08:00
Ciprian Hacman 602cb825e7 ARM64 support - Update bootstrap script for multi-arch 2020-06-19 04:42:11 +03:00
Ciprian Hacman 41defca8fe Detect supported architecture during node bootstrap 2020-06-03 17:17:32 +03:00
Peter Rifel fd17a118f5
Remove irrelevant TODO comment from userdata
This comment is from 2016, isn't clear, and I believe may be no longer relevant.

This will cleanup the TODO comment from everyone's userdata.
2020-04-18 08:54:35 -05:00
Justin Santa Barbara 6c41748640
Download nodeup into a writeable path on COS
/opt/ isn't writeable on COS, so we download to /var/lib/toolbox/kops
instead.
2020-01-04 18:16:08 -05:00
Ciprian Hacman 993b808cfc Move nodeup to /opt/kops/bin 2019-12-28 20:18:45 +02:00
Justin SB 5d8f680c12
nodeup download: try to use compression
We pass the appropriate flag to curl / wget, but where it fails we
fall back to not using compression.

Also simplify the code paths here, as it has proven so tricky - we
simply try a sequence of commands until we successful download and a
hash match.
2019-10-05 16:40:01 -04:00
mikesplain 9e55b8230a Update copyright notices
Also cleans some white spaces
2019-09-09 14:47:51 -04:00
Justin SB 461e78b4b3
Publish sha256 artifacts for kops itself 2019-08-25 17:28:13 -07:00
Kubernetes Prow Robot 675eeaa07b
Merge pull request #7447 from justinsb/use_mirrors
nodeup download should try all mirrors
2019-08-22 07:10:43 -07:00
Justin SB 05bbf9cbcc
bash script: don't assume nodeup filename
We were previously assuming that the name of the URL matched the name
we wanted to download to, but also assuming that the name of the file
was nodeup.  Change to download to a known file location, whatever the
mirror url.
2019-08-22 01:13:47 -04:00
Justin SB 2462ab1c02
nodeup download should try all mirrors
On an error, we were bailing out early!
2019-08-22 01:05:16 -04:00
Kashif Saadat 2756d58dbb Don't default adding MIMEBOUNDARY headers when a mixed instances policy is set 2019-08-02 11:12:05 +01:00
Rodrigo Menezes 869ed0e4d7 (WIP) Fix issue 6700: User Data for launch templates when using terraform. 2019-04-07 14:17:13 -07:00
Justin SB 26bd75aecb
Bulk spelling fixes
Experimenting with my own spelling checker, these are the typos it caught.
2018-12-20 17:43:56 -05:00
Justin Santa Barbara e6bf6b9f32 Field names are case-sensitive again
There was a regression in apimachinery which meant that kubernetes
tolerated field names with incorrect case.  Upstream bug is
https://github.com/kubernetes/kubernetes/issues/64612

Syncing up with latest kubernetes will mean we get the same breaking
change as kubernetes has/had.  It should only affect people that are
manually building YAML / JSON.

Added as a significant item to release notes.
2018-09-23 19:23:28 -04:00
Justin Santa Barbara 8a483c124e Avoid using which, CoreOS doesn't always have it
We just try executing `curl --version` instead, and fall back to wget.
We can't use `wget --version` because busybox wget doesn't support
`--version`.
2018-09-21 13:54:13 -04:00
Justin Santa Barbara c5c2bd1acb Don't repeatedly download nodeup
Only delete if the file doesn't match the hash.

Should help with retry issues / bandwidth
2018-07-19 12:09:13 -04:00
AdamDang b3ee3af4de
Update nodeup.go 2018-04-22 10:12:08 +08:00
Justin Santa Barbara e158f84e9f Set AWS_REGION into bootstrapscript
Fix #4451
2018-04-12 17:39:24 -04:00
andrewsykim 27e8902016 digitalocean: add nodeup support 2018-04-01 10:11:07 -04:00
Fred Dubois 6b38b2c3a6
Add support for bastion aws user-data
Fixes #4444
2018-02-19 08:52:10 -05:00
Justin Santa Barbara 86d18808fe Update bazel build files 2017-11-12 22:37:47 -05:00
Rodrigo Menezes ef24cec62b Fixing naming convention and adding running API machinery. 2017-11-09 14:30:48 -08:00
Rodrigo Menezes 3cfa67c3e6 Merge remote-tracking branch 'upstream/master' into extra_user-data 2017-11-08 22:20:54 -08:00
Rodrigo Menezes 1fe56a1603 Move user-data to IG spec 2017-11-06 23:25:37 -08:00
chrislovecnm 609e268a1d gazelle updates with new bazel version 2017-11-05 17:41:53 -07:00
Rodrigo Menezes e77cda1af1 Allow passing in extra user-data to cloud-init 2017-10-15 17:10:03 -07:00
Kubernetes Submit Queue 518e97d97b Merge pull request #3510 from justinsb/bazel
Automatic merge from submit-queue.

Initial bazel support

Builds on the 1.8 version bump

The "trick" is to strip the BUILD & BUILD.bazel files from the vendor-ed deps.

Will rebase after 1.8 version bump merges.
2017-10-03 01:19:27 -07:00
Justin Santa Barbara 0143be7c4f autogen: BUILD and BUILD.bazel 2017-10-02 14:27:21 -04:00
Justin Santa Barbara eb2cd45cdf Support wget for download, not just curl
Some images don't include curl but do have wget.  Prefer curl, but
fallback to wget.

(The official Debian 9 image does not have curl.)
2017-09-30 19:12:58 -04:00
Rohith 293292173a Inline Conponent Configuration
The current implementation does not ignore any possible interpolation of bash in the content. This PR wrapped the various spec content in 'EOF' to ignore all.

- updated the tests to reflect the changes
- wrapped the component configuration in 'eof' to ensure interpolation is ignored
2017-08-25 00:36:06 +01:00