fix: Allow to configure AI Copilot OpenAI endpoint when install via Helm#1548
Conversation
|
Welcome to the Microcks community! 💖 Thanks and congrats 🎉 for opening your first pull request here! Be sure to follow the pull request template or please update it accordingly. Hope you have a great time there! 🌟 ~~~~~~~~~ 🌟 📢 If you like Microcks, please ⭐ star ⭐ our repo to support it! 🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact. |
|
Thanks! Can you rebase your branch and start from 1.11.x? That's where development happens now. |
3735799 to
ec34c05
Compare
I have rebased the PR on 1.11.x. |
|
This is very close to be merged! Can you please review and fix the DCO check violations? Commits have to signed. This should be a matter of:
|
ec34c05 to
ec4b9c8
Compare
I have signed the commit , now all checks are passing |
lbroudoux
left a comment
There was a problem hiding this comment.
Sorry I looked at it again and realize the property name was wrong. Can you fix this?
api-url property can be passed as optional field for openai configuration Signed-off-by: Gopal Jha<[email protected]> Signed-off-by: Gopal Jha <[email protected]>
ec4b9c8 to
32422b8
Compare
|
You are now a Microcks community contributor! 💖 Thanks and congrats 🚀 on merging your first pull request! We are delighted and very proud of you! 👏 📢 If you're using Microcks in your organization, please add your company name to this list. 🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact. If you need to know why and how to add yourself to the list, please read the blog post "Join the Microcks Adopters list and Empower the vibrant open source Community 🙌" Kudos and please keep going, we need you 🙌 🌟 ~~~~~~~~~ 🌟 📢 If you like Microcks, please ⭐ star ⭐ our repo to support it! 🙏 It really helps the project to gain momentum and credibility. It's a small contribution back to the project with a big impact. |
Description
Related issue(s)
Fixes #1547