Description

Follow up to #201 .

It seems there were two bugs.

One was that the the inference profile seems to need to be added. Unclear why this worked locally earlier, but that is not important.

The other is that for the enterprise organization it seems that the ownership check is not needed. For now, we will simply require the approved for integ test label We can attempt to optimize if the collaborators find this provides too much friction.

Related Issues

#201

Type of Change

• Bug fix

Testing

[How have you tested the change?]

• hatch fmt --linter
• hatch fmt --formatter
• hatch test --all
• Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

Tested tests-integ/test_model_litellm.py

========================================================================= 1 passed, 54 warnings in 13.73s ==========================================================================

Checklist

• I have read the CONTRIBUTING document
• I have added tests that prove my fix is effective or my feature works
• I have updated the documentation accordingly
• I have added an appropriate example to the documentation to outline the feature
• My changes generate no new warnings
• Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.