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

Ingress playbook change #1090

Merged
merged 4 commits into from
Sep 21, 2023
Merged

Ingress playbook change #1090

merged 4 commits into from
Sep 21, 2023

Conversation

Avi-Robusta
Copy link
Contributor

@Avi-Robusta Avi-Robusta commented Sep 21, 2023

Screen Shot 2023-09-21 at 15 44 51

Needs change in frontend for new timeline, old timeline works well

Copy link
Contributor

@arikalon1 arikalon1 left a comment

Choose a reason for hiding this comment

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

very nice

@arikalon1 arikalon1 merged commit da201b8 into master Sep 21, 2023
14 checks passed
@arikalon1 arikalon1 deleted the ingress_playbook_change branch September 21, 2023 13:39
pavangudiwada pushed a commit to pavangudiwada/robusta that referenced this pull request Nov 6, 2023
* move ingress change to its own finding

* add ingress change aggregation key

* added option to override aggregation key

* renaming aggregation key
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants