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

Feature/dataplane inspection konvoyctl new #109

Merged
merged 12 commits into from
Aug 22, 2019

Conversation

jakubdyszkiewicz
Copy link
Contributor

@jakubdyszkiewicz jakubdyszkiewicz commented Aug 15, 2019

Somehow make generate generated some validation Protobuf files for existing files.

New core entity includes dataplane and dataplane insights and it results in dataplane inspection. New endpoint is /mesh/{mesh}/dataplane-inspections.

Sorry about the PR size.

@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/dataplane-inspection-konvoyctl-new branch from 39b9a71 to a25b69c Compare August 19, 2019 13:59
Copy link
Contributor

@yskopets yskopets left a comment

Choose a reason for hiding this comment

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

  1. Let's exclude *.pb.validate.go files from this PR
  2. Let's rename DataplaneInspection into DataplaneOverview

@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/dataplane-inspection-konvoyctl-new branch from 796d864 to 22e713d Compare August 21, 2019 16:32
@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/dataplane-inspection-konvoyctl-new branch from 22e713d to cb0c3c4 Compare August 21, 2019 16:39
Copy link
Contributor

@yskopets yskopets left a comment

Choose a reason for hiding this comment

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

A few minor changes

@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/dataplane-inspection-konvoyctl-new branch from 112829a to 0f0b7c3 Compare August 22, 2019 16:35
@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/dataplane-inspection-konvoyctl-new branch from 0f0b7c3 to 5c72464 Compare August 22, 2019 17:58
@jakubdyszkiewicz jakubdyszkiewicz merged commit 943b231 into master Aug 22, 2019
@jakubdyszkiewicz jakubdyszkiewicz deleted the feature/dataplane-inspection-konvoyctl-new branch August 22, 2019 18:41
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.

None yet

2 participants