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.

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

Nginx plus Ingress Regex #1656

Merged
merged 3 commits into from Jun 16, 2021
Merged

Nginx plus Ingress Regex #1656

merged 3 commits into from Jun 16, 2021

Conversation

SumoSourabh
Copy link
Contributor

Description

Adding regex for Nginx Plus Ingress

Testing performed
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

@SumoSourabh SumoSourabh requested a review from a team as a code owner June 14, 2021 15:04
@SumoSourabh SumoSourabh changed the title Nginx plus sourabh Nginx plus Ingress Regex Jun 14, 2021
@sumo-drosiek sumo-drosiek merged commit 574fcb5 into main Jun 16, 2021
@sumo-drosiek sumo-drosiek deleted the nginx_plus_sourabh branch June 16, 2021 05:25
@perk-sumo perk-sumo added this to the v2.2 milestone Jun 16, 2021
pmalek-sumo pushed a commit that referenced this pull request Jul 12, 2021
@SumoSourabh @pmalek
feat: add nginx plus to prometheus remote write (#1656)
01084a9 
* adding regex for Nginx Plus metrics.

* adding regex for Nginx Plus ingress metrics.
pmalek-sumo pushed a commit that referenced this pull request Jul 12, 2021
@pmalek @SumoSourabh
feat: add nginx plus to prometheus remote write (#1656) (#1685)
271910a 
* adding regex for Nginx Plus metrics.

* adding regex for Nginx Plus ingress metrics.

Co-authored-by: Sourabh Jain <45626610+SumoSourabh@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pmalek-sumo pmalek-sumo pmalek-sumo approved these changes

@kkujawa-sumo kkujawa-sumo kkujawa-sumo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.2
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@SumoSourabh @pmalek-sumo @kkujawa-sumo @sumo-drosiek @perk-sumo