alan-turing-institute / reginald

Reginald repository for REG Hack Week 23
3 stars 0 forks source link

Pulumi for hackweek #148

Closed rchan26 closed 7 months ago

rchan26 commented 8 months ago

Towards #113 and #127

Renamed 'production' stack to 'hackweek'. Changes:

Edit: I've also merged in pulumi-production here from #147 which renames the 'api-bot' stack to 'production'. This might seem confusing, but I think 'production' should refer to our main model which now the 'api-bot' and so that requires renaming the previous 'production' to 'hackweek'.

Similarly, updates to that model are:

Note that on the Azure container instances, we seem to get httpx.ConnectTimeout errors but not locally. Something to figure out in the future.

The PR also includes a few important dependency updates - the hardest being was to update openai to 1.6.1 from 0.2.7. Needed a few small changes to update llama-index too.

rwood-97 commented 7 months ago

Potential problem for users without prior access to the keyvault:

ERROR: (Forbidden) The user, group or application 'appid=04b07795-8ddb-461a-bbee-02f9e1bf7b46;oid=0fa34a95-4888-4614-9810-e912eb316082;numgroups=20;iss=https://sts.windows.net/4395f4a7-e455-4f95-8a9f-1fbaef6384f9/' does not have keys create permission on key vault 'kv-reginald-hackweek;location=uksouth'. For help resolving this issue, please see https://go.microsoft.com/fwlink/?linkid=2125287
Code: Forbidden
Message: The user, group or application 'appid=04b07795-8ddb-461a-bbee-02f9e1bf7b46;oid=0fa34a95-4888-4614-9810-e912eb316082;numgroups=20;iss=https://sts.windows.net/4395f4a7-e455-4f95-8a9f-1fbaef6384f9/' does not have keys create permission on key vault 'kv-reginald-hackweek;location=uksouth'. For help resolving this issue, please see https://go.microsoft.com/fwlink/?linkid=2125287
Inner error: {
    "code": "AccessDenied"
}

Fix for now is for someone with access to add new users.

rchan26 commented 7 months ago

thanks a lot @rwood-97 for your changes. will merge now!