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

feat: setup explorer on argus #886

Merged
merged 86 commits into from
Jun 20, 2024
Merged

feat: setup explorer on argus #886

merged 86 commits into from
Jun 20, 2024

Conversation

jakeyheath
Copy link
Contributor

@jakeyheath jakeyheath commented Apr 17, 2024

Summary

This is a demo PR of what the workflow and new configuration for the argus platform would look like. All new infra configuration is in the .infra folder and split by environment (very similar to how .happy looks). However, instead of a terraform files, there are YAML configuration files. These files look more like native kubernetes setting and load a Helm chart.

Details

  • .infra/common.yaml - Configuration for all environments. Think of this as your base set of common configuration.
  • .infra/rdev/values.yaml - Configuration overridees for the rdev environment. These settings can add or overwrite the configurations set in .infra/common.yaml.

References

  • The helm chart source is here.

Notes

  • There is an open bug in tiledb that isn't allowing us to use the pod's EKS IAM role. For now, we have manually made an AWS user and associated it with the policy and stored the credentials in the K8S secrets.

@jakeyheath jakeyheath requested a review from ebezzi April 17, 2024 17:42
Comment on lines 6 to 12
paths-ignore:
- '.github/**'
- '.happy/**'
- 'dev_docs/**'
- 'example-dataset/**'
branches-ignore:
- main
Copy link
Contributor Author

Choose a reason for hiding this comment

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

  • only rebuild if the source changes
  • don't build and push on the main branch (the container image should already be built by the time it merges to main)

@@ -1,4 +1,4 @@
FROM ubuntu:22.04
FROM 471112759938.dkr.ecr.us-west-2.amazonaws.com/docker.io/sci/library/ubuntu:22.04
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Utilizes our ECR pull through cache so we don't accidentally hit dockerhub rate limit issus. Also allows us to do security scanning and other security monitoring on the images we pull.

.infra/common.yaml Outdated Show resolved Hide resolved
@hspitzley-czi hspitzley-czi added stack and removed stack labels Jun 17, 2024
.infra/common.yaml Outdated Show resolved Hide resolved
.infra/common.yaml Outdated Show resolved Hide resolved
.infra/common.yaml Outdated Show resolved Hide resolved
@hspitzley-czi hspitzley-czi added stack and removed stack labels Jun 17, 2024
@hspitzley-czi hspitzley-czi marked this pull request as ready for review June 18, 2024 21:09
@hspitzley-czi hspitzley-czi merged commit a6ecee7 into main Jun 20, 2024
16 of 17 checks passed
@hspitzley-czi hspitzley-czi deleted the heathj/argus branch June 20, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants