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

feat: integrate pipelines API #2457

Merged
merged 26 commits into from
Jul 31, 2023
Merged

feat: integrate pipelines API #2457

merged 26 commits into from
Jul 31, 2023

Conversation

nityanandagohain
Copy link
Member

Please go through #2456 first.

Fixes #1779

This PR, integrates the rest API with opamp

@github-actions github-actions bot added the chore label Mar 15, 2023
@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@nityanandagohain nityanandagohain changed the title chore: integrate pipelines API feat: integrate pipelines API Mar 15, 2023
@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions github-actions bot added the enhancement New feature or request label Mar 16, 2023
@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

Base automatically changed from feat/opamp-logparing to develop March 22, 2023 04:25
@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@nityanandagohain
Copy link
Member Author

Please have a look at this PR.

srikanthccv
srikanthccv reviewed Mar 23, 2023
View reviewed changes
pkg/query-service/app/http_handler.go Outdated Show resolved Hide resolved
pkg/query-service/app/http_handler.go Outdated Show resolved Hide resolved
pkg/query-service/app/http_handler.go Outdated Show resolved Hide resolved
pkg/query-service/app/http_handler.go Outdated Show resolved Hide resolved
pkg/query-service/app/http_handler.go Outdated Show resolved Hide resolved
pkg/query-service/app/logparsingpipeline/controller.go Outdated Show resolved Hide resolved
pkg/query-service/app/logparsingpipeline/controller.go Outdated Show resolved Hide resolved
pkg/query-service/app/logparsingpipeline/controller.go Outdated Show resolved Hide resolved
pkg/query-service/app/logparsingpipeline/db.go Outdated Show resolved Hide resolved
pkg/query-service/app/logparsingpipeline/postablePipeline.go Outdated Show resolved Hide resolved
@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 8 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@CLAassistant
Copy link

CLAassistant commented Apr 21, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@srikanthccv
Copy link
Member

srikanthccv commented Jul 31, 2023 •
edited
Loading

Please resolve conflicts. As discussed in Slack, I will the approval to get merged to develop but will review again after the merge.

@raj-k-singh raj-k-singh force-pushed the feat/logparsing-integration branch 2 times, most recently from ba23cc8 to 48ba2f3 Compare July 31, 2023 15:14
@raj-k-singh raj-k-singh force-pushed the feat/logparsing-integration branch from 48ba2f3 to 68a14f3 Compare July 31, 2023 15:19
@raj-k-singh raj-k-singh merged commit 1a3e46c into develop Jul 31, 2023
@raj-k-singh raj-k-singh deleted the feat/logparsing-integration branch July 31, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@raj-k-singh raj-k-singh raj-k-singh left review comments

@srikanthccv srikanthccv srikanthccv approved these changes

@ankitnayan ankitnayan Awaiting requested review from ankitnayan

@makeavish makeavish Awaiting requested review from makeavish

Assignees
No one assigned
Labels
chore codeball:needs-careful-review Codeball enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enable log parsing from the UI
5 participants
@nityanandagohain @CLAassistant @srikanthccv @raj-k-singh @makeavish