Skip to content

[RPC] Revert "Implemented rpc logging (#10967)"#11227

Merged
manupak merged 1 commit into
apache:mainfrom
leandron:revert_10967
May 6, 2022
Merged

[RPC] Revert "Implemented rpc logging (#10967)"#11227
manupak merged 1 commit into
apache:mainfrom
leandron:revert_10967

Conversation

@leandron

@leandron leandron commented May 6, 2022

Copy link
Copy Markdown
Contributor

This reverts commit aa3bcd9, because it fails on Windows CI as reported in issue #11220. PR #11223 tries to address it but is is failing in the regular CI with testing issue on Hexagon.

I think the fix to this one is not something we can just quickly implement, so I'm reverting this to unblock CI. I suggest a new PR can be submitted with a combination of #10967 and #11223 with a working version of this feature that passes Windows CI.

cc @mehrdadh @mkatanbaf @Mousius @u99127 for reviews and quick merge to unblock CI

This reverts commit aa3bcd9, because it
fails on Windows CI as reported in issue apache#11220. PR apache#11223 tries to address
it but is is failing in the regular CI with testing issue on Hexagon.
@github-actions github-actions Bot requested review from Mousius and mehrdadh May 6, 2022 09:07

@Mousius Mousius left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good call @leandron, let's get back the pipeline back to action 😸

@manupak manupak merged commit ff7efe7 into apache:main May 6, 2022
@manupak

manupak commented May 6, 2022

Copy link
Copy Markdown
Contributor

This is merged in favor of unblocking the CI...

@areusch

areusch commented May 6, 2022

Copy link
Copy Markdown
Contributor

sorry for the breakage everyone! we'll fix this and push a new PR.

shtinsa pushed a commit to Deelvin/tvm that referenced this pull request May 17, 2022
This reverts commit aa3bcd9, because it
fails on Windows CI as reported in issue apache#11220. PR apache#11223 tries to address
it but is is failing in the regular CI with testing issue on Hexagon.
SebastianBoblest pushed a commit to SebastianBoblest/tvm that referenced this pull request May 27, 2022
This reverts commit aa3bcd9, because it
fails on Windows CI as reported in issue apache#11220. PR apache#11223 tries to address
it but is is failing in the regular CI with testing issue on Hexagon.
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.

4 participants