Skip to content

Update codeowners for ess_billing package #15317

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Oct 22, 2025
Merged

Update codeowners for ess_billing package #15317

merged 4 commits into from
Oct 22, 2025

Conversation

@lalit-satapathy
Copy link
Contributor

@lalit-satapathy lalit-satapathy commented Sep 15, 2025

Update the code owners for ess_billing package, as discussed offline.

agithomas
agithomas approved these changes Sep 15, 2025
View reviewed changes
Copy link
Contributor

@agithomas agithomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

3kt
3kt approved these changes Sep 15, 2025
View reviewed changes
Copy link
Contributor

@3kt 3kt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Co ownership, love this <3
Thank you for leaving our brand in there :)

@lalit-satapathy lalit-satapathy requested a review from a team as a code owner September 15, 2025 12:30
@lalit-satapathy
Copy link
Contributor Author

lalit-satapathy commented Sep 15, 2025

Also updated the manifest here.

agithomas
agithomas reviewed Sep 15, 2025
View reviewed changes
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Sep 15, 2025

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elastic-sonarqube
Copy link

elastic-sonarqube bot commented Sep 15, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

3kt
3kt approved these changes Sep 15, 2025
View reviewed changes
Copy link
Contributor

@3kt 3kt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

packages/ess_billing/manifest.yml
owner:
github: elastic/customer-architects
type: community
github: elastic/obs-infraobs-integrations
Copy link
Contributor

@3kt 3kt Sep 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we can only have a single owner here?

Copy link
Member

@ishleenk17 ishleenk17 Sep 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats right.

Copy link
Contributor Author

@lalit-satapathy lalit-satapathy Sep 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes.

@andrewkroh andrewkroh added the Integration:ess_billing Elasticsearch Service Billing label Sep 15, 2025
ishleenk17
ishleenk17 approved these changes Sep 16, 2025
View reviewed changes
packages/ess_billing/manifest.yml
owner:
github: elastic/customer-architects
type: community
github: elastic/obs-infraobs-integrations
Copy link
Member

@ishleenk17 ishleenk17 Sep 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats right.

@lalit-satapathy
Copy link
Contributor Author

lalit-satapathy commented Sep 16, 2025

fyi: @lucabelluccini

@lucabelluccini
Copy link
Contributor

lucabelluccini commented Sep 16, 2025

Thank you @lalit-satapathy & @daniela-elastic

@botelastic
Copy link

botelastic bot commented Oct 16, 2025

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Oct 16, 2025
@lucabelluccini
Copy link
Contributor

lucabelluccini commented Oct 20, 2025

Should this be merged?

@botelastic botelastic bot removed the Stalled label Oct 20, 2025
@elasticmachine
Copy link

elasticmachine commented Oct 22, 2025

💚 Build Succeeded

History

@lalit-satapathy lalit-satapathy merged commit dbf1e3e into elastic:main Oct 22, 2025
7 checks passed
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Oct 22, 2025

Package ess_billing - 1.6.0 containing this change is available at https://epr.elastic.co/package/ess_billing/1.6.0/

alexreal1314 pushed a commit to alexreal1314/integrations that referenced this pull request Oct 22, 2025
@lalit-satapathy @alexreal1314
Update codeowners for ess_billing package (elastic#15317)
96bb9f1 
* Update codeowners for ess_billing
agithomas pushed a commit to agithomas/integrations that referenced this pull request Oct 30, 2025
@lalit-satapathy @agithomas
Update codeowners for ess_billing package (elastic#15317)
d6421ff 
* Update codeowners for ess_billing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agithomas agithomas agithomas approved these changes

@3kt 3kt 3kt approved these changes

@ishleenk17 ishleenk17 ishleenk17 approved these changes

@daniela-elastic daniela-elastic daniela-elastic approved these changes

@tommyers-elastic tommyers-elastic Awaiting requested review from tommyers-elastic

Assignees

No one assigned

Labels

Integration:ess_billing Elasticsearch Service Billing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@lalit-satapathy @lucabelluccini @elasticmachine @3kt @agithomas @ishleenk17 @daniela-elastic @andrewkroh