Skip to content

Create Issue with more clear title and body#18374

Merged
phlax merged 16 commits intoenvoyproxy:mainfrom
ME-ON1:create-issue-dep
Oct 5, 2021
Merged

Create Issue with more clear title and body#18374
phlax merged 16 commits intoenvoyproxy:mainfrom
ME-ON1:create-issue-dep

Conversation

@ME-ON1
Copy link
Copy Markdown
Contributor

@ME-ON1 ME-ON1 commented Oct 1, 2021

Signed-off-by: Tarun Sharma starun.1998@gmail.com

Following up for the #17876 (comment)

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
@phlax phlax self-assigned this Oct 1, 2021
ME-ON1 added 4 commits October 2, 2021 22:34
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
@ME-ON1
Copy link
Copy Markdown
Contributor Author

ME-ON1 commented Oct 2, 2021

Hi phlax i have create both more clear issue body message and close old package issue in this PR.

@tederigobert

This comment has been minimized.

@tederigobert

This comment has been minimized.

@ME-ON1
Copy link
Copy Markdown
Contributor Author

ME-ON1 commented Oct 3, 2021

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #18374 (comment) was created by @ME-ON1.

see: more, trace.

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

first pass and this is looking good, thanks @ME-ON1

feedback inline...

@phlax
Copy link
Copy Markdown
Member

phlax commented Oct 3, 2021

for reference here are the test issues i created https://github.com/phlax/envoy/issues

ME-ON1 added 5 commits October 5, 2021 00:02
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
ME-ON1 added 2 commits October 5, 2021 21:49
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

one small nit and then i reckon this is ready to land

thanks @ME-ON1

Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Signed-off-by: Tarun Sharma <starun.1998@gmail.com>
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @ME-ON1

and thanks again for working on this, this tool is already incredibly useful

@phlax phlax merged commit f867d3a into envoyproxy:main Oct 5, 2021
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.

3 participants