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.

Sign up for GitHub

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

Jump to bottom

Harjit managed app #241

Open
wants to merge 29 commits into
base: main
Choose a base branch
from
Open

Harjit managed app #241

wants to merge 29 commits into from

Conversation

harjitdotsingh
Copy link

Update the APP to deploy Helm Chart and add openapispec for the frontend

jgbradley1 and others added 28 commits January 2, 2025 13:44
@jgbradley1
refactor of bicep variable names to be more generic and pytest cleanup
98fc3b8
@jgbradley1
add RBAC reader role assignment to cosmosdb bicep deployment
2b41976
@jgbradley1
update dependabot settings
699bfa5
@jgbradley1
update code references to new locations in graphrag library
ff5714a
@jgbradley1
refactor variable names to be more generic and add integration tests
0252646
@jgbradley1
add new pytests
fbbad71
@jgbradley1
ruff format updates
2f744c1
@jgbradley1
revert bicep api version changes to a working condition
4e1b1ba
@jgbradley1
fix bad import
44d7859
@jgbradley1
temporary move of import statement
97d8b52
@jgbradley1
update synthetic index dataset
3c96059
@jgbradley1
refactor bicep to be cleaner and remove ssh public key generation for…
96dcd95 
… AKS deployment - no longer needed
@jgbradley1
update common bicep variable name
8d448f8
@jgbradley1
add aoai to bicep deployment
863bff3
@jgbradley1
cleanup bicep variable names
f068846
@jgbradley1
cleanup optional parameters and define default values in helm chart
743d587
@jgbradley1
auto-format bicep code
b6153f3
@jgbradley1
auto format code
5f78d37
@jgbradley1
fix aoai deployment issue
bc24bfa
@jgbradley1
working version of a managed app
ca0c489
@jgbradley1
Merge branch 'main' into joshbradley/convert-to-managed-app
25be69f
@jgbradley1
fix changes after merge with main
24df161
@jgbradley1
fix synthetic dataset after merge with main
871b192
@harjitdotsingh
updated for build
82871df
@harjitdotsingh
updated for build
964b536
@harjitdotsingh
ManagedAPP
6364389
@harjitdotsingh
Updated URL
b687cc0
@harjitdotsingh
updated README
a4dede0
@harjitdotsingh harjitdotsingh requested a review from a team as a code owner February 12, 2025 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timothymeyers timothymeyers timothymeyers left review comments

@jgbradley1 jgbradley1 Awaiting requested review from jgbradley1

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@harjitdotsingh @timothymeyers @jgbradley1