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

repokitteh: implement retest azp #12402

Merged
merged 4 commits into from
Aug 7, 2020
Merged

repokitteh: implement retest azp #12402

merged 4 commits into from
Aug 7, 2020

Conversation

lizan
Copy link
Member

@lizan lizan commented Jul 31, 2020

Signed-off-by: Lizan Zhou [email protected]

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/rk:bless

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/rk-bless

@repokitteh-read-only
Copy link

This PR has been blessed, meaning its repokitteh changes are in effect in this PR.

😽

Caused by: #12402 was labeled by repokitteh[bot].

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: github.com/envoyproxy/envoy/ci/repokitteh/modules/azure_pipelines.star:14:5: cannot reassign global _retry declared at github.com/envoyproxy/envoy/ci/repokitteh/modules/azure_pipelines.star:5:5
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/rk-bless

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: github.com/envoyproxy/envoy/ci/repokitteh/modules/azure_pipelines.star:10:37: undefined: config
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/rk-bless

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: struct has no .get_check_runs attribute
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/rk-bless

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp

@repokitteh-read-only
Copy link

["call", "create_status", "get_combined_statuses", "issue_assign", "issue_check_assignee", "issue_create_comment", "issue_create_comment_reaction", "issue_label", "issue_list_comments", "issue_list_labels", "issue_unassign", "issue_unlabel", "pr_get_review", "pr_get_review_comments", "pr_list_files", "pr_list_reviewers", "pr_list_reviews", "pr_request_review", "pr_unrequest_review", "repo_is_collaborator", "team_get_by_name", "team_list_members"]

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp 46718

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: github.com/envoyproxy/envoy/ci/repokitteh/modules/azure_pipelines.star:16:15: undefined: comment_id
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp 46719

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: function http error: headers: unexpected type: got *map[string]string, want map[interface {}]interface {}
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp

@repokitteh-read-only
Copy link

TODO: please specify build id

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: list index 0 out of range [0:0]
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Jul 31, 2020

/retry-azp 46720

@repokitteh-read-only
Copy link

[]

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@dio dio assigned itayd Aug 1, 2020
@lizan
Copy link
Member Author

lizan commented Aug 6, 2020

/retest

@repokitteh-read-only
Copy link

🙀 Error while processing event:

evaluation error
error: local variable msgs referenced before assignment:
 Traceback (most recent call last):
  bootstrap:143: in <toplevel>
  bootstrap:135: in _main
  bootstrap:62: in _call
  bootstrap:15: in _call1
  github.com/envoyproxy/envoy/ci/repokitteh/modules/azure_pipelines.star:27: in _retry
Error: local variable msgs referenced before assignment
😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Aug 6, 2020

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines, to retry CircleCI checks, use /retest-circleCannot retry non-completed check: envoy-presubmit, please wait.

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
WIP: repokitteh azp retest
1df1310 
Signed-off-by: Lizan Zhou <[email protected]>
@lizan
Copy link
Member Author

lizan commented Aug 6, 2020

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines, to retry CircleCI checks, use /retest-circle.
Cannot retry non-completed check: envoy-presubmit, please wait.

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Aug 6, 2020

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines, to retry CircleCI checks, use /retest-circle.
Retried failed jobs in: envoy-presubmit

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan lizan changed the title WIP: repokitteh azp retest repokitteh: implement retest azp Aug 6, 2020
@lizan
Copy link
Member Author

lizan commented Aug 6, 2020

@itayd see the 4 comments above, this is ready to go. PTAL

itayd
itayd approved these changes Aug 6, 2020
View reviewed changes
Copy link
Contributor

@itayd itayd left a comment

Choose a reason for hiding this comment

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

Awesome. :)

FYI @htuch @mattklein123

itayd
itayd previously approved these changes Aug 6, 2020
View reviewed changes
@lizan lizan requested a review from mattklein123 August 6, 2020 23:02
jmarantz
jmarantz previously approved these changes Aug 6, 2020
View reviewed changes
dio
dio previously approved these changes Aug 6, 2020
View reviewed changes
ci/repokitteh/modules/azure_pipelines.star Outdated
_azp_context_prefix = "ci/azp: "

def _retry_azp(organization, project, build_id, token):
"""Makes an Azupre Pipelines Build API request"""
Copy link
Member

Choose a reason for hiding this comment

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

nit. s/Azupre/Azure

@lizan
fix
5642e64 
Signed-off-by: Lizan Zhou <[email protected]>
@lizan lizan dismissed stale reviews from dio, jmarantz, and itayd via 5642e64 August 6, 2020 23:39
@lizan
Copy link
Member Author

lizan commented Aug 7, 2020

/retest-circle

@repokitteh-read-only
Copy link

🐴 hold your horses - no failures detected, yet.

😽

Caused by: a #12402 (comment) was created by @lizan.

see: more, trace.

@lizan
Copy link
Member Author

lizan commented Aug 7, 2020

@mattklein123 can you force merge this? I don't know why circle is stuck here

@mattklein123
Copy link
Member

Sweet!

@mattklein123 mattklein123 merged commit fdab03e into envoyproxy:master Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dio dio dio approved these changes

@jmarantz jmarantz jmarantz left review comments

@itayd itayd itayd left review comments

@mattklein123 mattklein123 Awaiting requested review from mattklein123

Assignees

@itayd itayd

Labels
rk:blessed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lizan @mattklein123 @dio @itayd @jmarantz