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

Create AddOn API #280

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Create AddOn API #280

wants to merge 1 commit into from

Conversation

zugao
Copy link
Collaborator

@zugao zugao commented Dec 5, 2024

Summary

  • This PR changes API for how we add add-ons
  • This PR fixes the problem with billing that the instance id of an add-on is the same as the parent service.

Context

Billing

We currently have a problem with billing as we are sending the same instance_id for service and it's add-on office(collabora). I order to fix this issue we decided to have a dedicated place for AddOns where we can easily track what addOns are available. This object will reside under spec.parameters. Once we know where the add ons objects are, we can easily track and use them across our comp-functions, including the billing. We also change the naming from collabora to office, as this is how it's defined in Odoo and it's also simpler to understand the add from the end user perspective.

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.
  • Link this PR to related issues.

@zugao zugao added the minor label Dec 5, 2024
@zugao zugao requested review from a team, Kidswiss, TheBigLee and wejdross and removed request for a team December 5, 2024 12:16
@wejdross
Copy link
Member

wejdross commented Dec 5, 2024

Can You explain this PR better? How is that going to help with billing? Why introducing Addons? - To generate prometheus rule based on that? Why naming addon as 'office' instead of 'collabora'?

@zugao zugao marked this pull request as draft December 5, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants