UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects
Merged Andrew Kesterson requested to merge 577th_enablement into master

General MR

Summary

This corrects 3 problems discovered while helping a recent customer onboard with the quickstart:

#2591 (closed) : Quickstart instructions are ambiguous regarding where some commands should be run, exactly what the big bang cluster will have running on it, and how to restart the process if cluster creation fails. Also calls out a known incompatibility with yq versions < 4.45.1.

#2592 (closed) : k3d-dev has a hardcoded reference to the ubuntu username in it, foiling some users

#2593 (closed) : quickstart.sh --destroy doesn't handle user provided arguments properly

Relevant logs/screenshots

Hardcoded ubuntu username reference

Before

quickstart.sh -U myusername
# ... snip ...
+ sudo usermod -aG docker ubuntu
usermod: user 'ubuntu' does not exist

After

quickstart.sh -U myusername
# ... snip ...
+ sudo usermod -aG docker myusername

quickstart processing --destroy

The --destroy flag is intended to completely destroy an instance, so the k3d-dev script can't actually process it on an instance it didn't create in AWS. But in this case it should fail gracefully, rather than failing with an error. This patch fixes the failure to be graceful and informative.

Before

$ bash quickstart.sh --destroy -H 192.168.1.1 -K ~/.ssh/demo.pem
An error occurred (UnauthorizedOperation) when calling the DescribeVpcs operation: You are not authorized to perform this operation. User:arn:aws:iam:*:user/* is not authorized to perform: ec2:DescribeVpcs because no identity-based policy allows the ec2:DescribeVpcs action
AWS account has no default VPC - please provide VPC_ID

After

$ bash quickstart.sh --destroy -H 192.168.1.1 -K ~/.ssh/demo.pem
The current instance may not have been provisioned by k3d-dev. It is probably not appropriate 
for k3d-dev to attempt to destroy it.
Nothing to do.

Linked Issue

Closes #2591 (closed)

Closes #2592 (closed)

Closes #2593 (closed)

Upgrade Notices

N/A

Edited by Andrew Kesterson

Merge request reports

Merge request pipeline #4058040 passed

Merge request pipeline passed for 4005f672

Merged by Michael MartinMichael Martin 3 weeks ago (Mar 28, 2025 6:00pm UTC)

Loading

Pipeline #4058623 passed

Pipeline passed for 87680696 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading