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(applications): add replibot airgap environment demo app #21

Closed
wants to merge 7 commits into from

Conversation

DexterYan
Copy link
Member

@DexterYan DexterYan commented Dec 3, 2024

  • terraform to provision an airgapped environment in aws
  • github action to build docker image and airgap bundle
  • helm chart
  • customer metric code example

README
demo app with github action
https://github.com/DexterYan/replicated-airgap-embedded-cluster-deploy-flow/actions/runs/12225567658

@DexterYan DexterYan marked this pull request as draft December 3, 2024 20:59
@DexterYan DexterYan marked this pull request as ready for review January 3, 2025 03:27
@DexterYan DexterYan changed the title WIP feat(applications): add replibot feat(applications): add replibot airgap environment demo app Jan 3, 2025
Copy link
Member

@xavpaice xavpaice left a comment

Choose a reason for hiding this comment

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

This isn't the Replibot application, and it needs to be really clear about that. We need a different name, that is more descriptive. I can see where a bunch of the original code has been pulled out but it's not complete, and needs a fresh name. So much of the original app is missing now that this might as well be a fresh project, without the risk of carryover from the prior version.

Please check the directory structure, it looks like there's some duplication in charts/replibot/replibot.

The README, and other descriptive test, needs to be cleaned up.

@@ -0,0 +1,28 @@
# Replibot
CRE tooling to interface with AI models for support info
This Repo includes the core service run receive github issue event and slack command event, and the support service will summary or search based on LLM.
Copy link
Member

Choose a reason for hiding this comment

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

I thought we cut it down to avoid all this?


# How to deploy the service

The service is automatically deployed by the github action, and the service is deployed to the replicated eks cluster.
Copy link
Member

Choose a reason for hiding this comment

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

no it isn't

"github.com/replicatedhq/replibot/api/param"
)

func RequiresAuthTokens() gin.HandlerFunc {
Copy link
Member

Choose a reason for hiding this comment

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

this isn't needed for this application

@@ -0,0 +1,28 @@
apiVersion: v2
name: replibot
description: A Helm chart for Kubernetes
Copy link
Member

Choose a reason for hiding this comment

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

this isn't descriptive at all

replicaCount: 1

image:
repository: wti2000/replibot-testing
Copy link
Member

Choose a reason for hiding this comment

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

this shouldn't be your personal repo

@DexterYan DexterYan closed this Jan 19, 2025
@DexterYan DexterYan deleted the dx/sc-116413/replibot-example branch January 19, 2025 21:46
@DexterYan
Copy link
Member Author

branch deleted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants