Skip to content

Commit

Permalink
Merge pull request #3842 from yuvipanda/no-mo-cloud-user
Browse files Browse the repository at this point in the history
Remove use of cloud-user-sa from openscapes hub
  • Loading branch information
yuvipanda committed Mar 25, 2024
2 parents 153d47c + cfd5ac1 commit ee53782
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 16 deletions.
1 change: 0 additions & 1 deletion config/clusters/openscapes/common.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ basehub:
name: NASA Openscapes
url: https://nasa-openscapes.github.io/
singleuser:
serviceAccountName: cloud-user-sa
defaultUrl: /lab
extraEnv:
GH_SCOPED_CREDS_CLIENT_ID: "Iv1.6981e043b45f042f"
Expand Down
15 changes: 0 additions & 15 deletions eksctl/openscapes.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,6 @@ local clusterRegion = "us-west-2";
local masterAzs = ["us-west-2a", "us-west-2b", "us-west-2c"];
local nodeAz = "us-west-2b";

// List of namespaces where we have hubs deployed
// Each will get a ServiceAccount that will get credentials to talk
// to AWS services, via https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts.html
local namespaces = ['staging', 'prod'];

// Node definitions for notebook nodes. Config here is merged
// with our notebook node definition.
// A `node.kubernetes.io/instance-type label is added, so pods
Expand Down Expand Up @@ -60,16 +55,6 @@ local daskNodes = [
availabilityZones: masterAzs,
iam: {
withOIDC: true,

serviceAccounts: [{
metadata: {
name: "cloud-user-sa",
namespace: namespace
},
attachPolicyARNs:[
"arn:aws:iam::aws:policy/AmazonS3FullAccess"
],
} for namespace in namespaces],
},
// If you add an addon to this config, run the create addon command.
//
Expand Down

0 comments on commit ee53782

Please sign in to comment.