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

Update to ACI template & Repackaging Helm Chart for release #14

Merged
merged 4 commits into from
Mar 8, 2023

Conversation

tawalke
Copy link
Contributor

@tawalke tawalke commented Mar 8, 2023

Purpose

  • Repackaging Helm chart for release of Qdrant to AKS
  • Fixing Storage file name length Issue with ACI ARM template.

Does this introduce a breaking change?

[ ] Yes
[ X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ X ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

For AKS helm chart update

  • Execute the AKS Deploy to Azure button to create your Kubernetes cluster
  • Clone repo git clone "This repo url"
  • Got to Azure-Kubernetes directory `
    cd ./Azure-Kubernetes-Svc/qdrant-on-azure
  • Execute command
    helm install <your installation name> ../qdrant-on-azure/ --create-namespace testing

For ACI template Change

  • Clone repo and run command to deploy new template from updated README.md instructions in ACI

What to Check

Verify that the following are valid

  • ...

Other Information

@tawalke tawalke requested a review from mcollier March 8, 2023 04:03
@tawalke
Copy link
Contributor Author

tawalke commented Mar 8, 2023

@mcollier Repackaged Helm chart, please test and review that you can deploy.

@tawalke tawalke changed the title Repackaging Helm Chart for release Update to ACI template & Repackaging Helm Chart for release Mar 8, 2023
@tawalke
Copy link
Contributor Author

tawalke commented Mar 8, 2023

@mcollier Merging PR to address ACI template typo.

Please review new AKS Helm package and let me know if you can deploy per instructions above.

@tawalke tawalke merged commit 27eae0e into main Mar 8, 2023
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.

1 participant