-
Notifications
You must be signed in to change notification settings - Fork 64
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
DRIVERS-2882 Add scripts to test AKS #453
Conversation
@@ -31,19 +31,6 @@ export AZUREKMS_TENANTID=$AZUREOIDC_TENANTID | |||
export AZUREKMS_SECRET=$AZUREOIDC_SECRET | |||
export AZUREKMS_CLIENTID=$AZUREOIDC_APPID | |||
|
|||
# Check for Azure Command-Line Interface (`az`) version 2.25.0 or newer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This block was moved to the login file.
@@ -0,0 +1,9 @@ | |||
## OIDC on K8S | |||
|
|||
- Launch an Atlas cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a placeholder for now, I still need to add GKE and EKS
@@ -47,19 +47,6 @@ done | |||
# Set defaults. | |||
export AZUREKMS_IMAGE=${AZUREKMS_IMAGE:-"Debian:debian-11:11:0.20221020.1174"} | |||
|
|||
# Check for Azure Command-Line Interface (`az`) version 2.25.0 or newer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This moved to the login file.
# Skip the "Processing triggers for man-db" step. | ||
echo "set man-db/auto-update false" | sudo debconf-communicate; sudo dpkg-reconfigure -f noninteractive man-db | ||
sudo apt-get -qq update | ||
OPTIONS="-y -qq -o DPkg::Lock::Timeout=-1" | ||
OPTIONS="-qq -y -o DPkg::Lock::Timeout=-1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cleaned up a bunch of these apt-get calls to actually be quiet
URL="" | ||
|
||
case $NAME in | ||
kubectl) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll be adding other binaries to this list in further PRs: gcloud and eksctl
@@ -0,0 +1,7 @@ | |||
# | |||
|
|||
## AKS Cluster Setup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again this is a stub for now
pip install -U -q requests | ||
pip install . | ||
popd | ||
echo "Installing dependencies ... end" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this use pip install -r requirements/<file_name>.txt
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually we don't need requests for this one, removed that part.
|
||
SCRIPT_DIR=$(dirname ${BASH_SOURCE[0]}) | ||
. $SCRIPT_DIR/../../handle-paths.sh | ||
pushd $SCRIPT_DIR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No matching popd
in this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed all three
|
||
SCRIPT_DIR=$(dirname ${BASH_SOURCE[0]}) | ||
. $SCRIPT_DIR/../../handle-paths.sh | ||
pushd $SCRIPT_DIR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No matching popd
in this file.
|
||
SCRIPT_DIR=$(dirname ${BASH_SOURCE[0]}) | ||
. $SCRIPT_DIR/../handle-paths.sh | ||
pushd $SCRIPT_DIR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No matching popd
in this file.
No description provided.