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 content understanding to private endpoint #2225

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pamelafox
Copy link
Collaborator

Purpose

Fixes #2218

We were missing the Content Understanding resource ID for the private endpoint configuration, so this PR adds it in.

I do not currently have a private deployment/ VPN setup, so I haven't been able to verify this fix personally. However, it's the same code we use for the other cognitive service accounts.

Does this introduce a breaking change?

When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.

[ ] Yes
[X] No

Does this require changes to learn.microsoft.com docs?

This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.

[ ] Yes
[X] No

Type of change

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

Code quality checklist

See CONTRIBUTING.md for more details.

N/A

@mrisahoo1
Copy link

Hi Pamela,
Thank you for your response. I tried it again with the One change you suggested in the main.bicep file.
Is there any other changes I need to make ?
But I am getting the same error still:
Traceback (most recent call last):
File "C:\Users\User1\Desktop\search-mco\app\backend\prepdocs.py", line 439, in
loop.run_until_complete(main(ingestion_strategy, setup_index=not args.remove and not args.removeall))
File "C:\Users\WKK9OUQ\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 650, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "C:\Users\User1\Desktop\search-mco\app\backend\prepdocs.py", line 242, in main
await strategy.setup()
File "C:\Users\User1\Desktop\search-mco\app\backend\prepdocslib\filestrategy.py", line 91, in setup
await cu_manager.create_analyzer()
File "C:\Users\User1\Desktop\search-mco\app\backend\prepdocslib\mediadescriber.py", line 78, in create_analyzer
raise Exception("Error creating analyzer", data)
Exception: ('Error creating analyzer', '{"error":{"code":"404","message": "Resource not found"}}')

@mrisahoo1
Copy link

Also, I have deployed the service of Container Apps for hosting the Application, do I have to do it in App Service instead of Container Apps ?

@mrisahoo1
Copy link

@pamelafox Is there any update, since the same error is coming. Even after adding it to the private endpoint.
The Content Understanding service is in the allowed regions as well.

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.

Errors while adding gpt-vision in the application and ingestion with Content Understanding.
2 participants