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

Add Helm Chart as an installation method #931

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

agentmilindu
Copy link
Collaborator

@agentmilindu agentmilindu commented Aug 6, 2020

Description

This is to make Bassa available as a Helm chart for users who are interested in installing Bassa on their Kubernetes clusters.

Motivation and Context

This will be an easy way forward for people who has a local/on-prem Kubernetes cluster and want to have a quick setup of Bassa running on them.

Users should be able to instal Bassa with the usual Helm commands,

helm install my-bassa bassa

How Has This Been Tested?

Hosted on a real Kubernetes cluster and Ingress was used to expose it to the internet.
URL: http://bassa.installer.to/#!/login ( This might not work when the temporary Kubernetes cluster expires )

Screenshots:

Screenshot from 2020-08-06 09-14-22

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Related Issue

Following issues need to be solved for this PR to be complete.

  1. Helm Chart for Bassa installation #930
  2. Move out Passwords and other ENV variables of Deployment.yamls. #926
  3. API DNS is hard coded in nginx.config #928
  4. Bassa UI is hardcoded to send the request to 'localhost:5000/api' #929

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@TravisBuddy
Copy link

Hey @agentmilindu,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: 3e130230-d79a-11ea-85fc-4b61cecd2c91

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