Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Conversation

@mseth10
Copy link
Contributor

@mseth10 mseth10 commented Feb 23, 2022

Description

Due to a breaking change in one of the dependecy pacakges markupsafe, we are getting this error
ImportError: cannot import name 'soft_unicode' from 'markupsafe'
in the website build pipeline https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-website-build-master/detail/restricted-website-build-master/3553/pipeline

More information about this issue: pallets/markupsafe#284

@mxnet-bot
Copy link

Hey @mseth10 , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [sanity, edge, clang, miscellaneous, website, windows-cpu, windows-gpu, centos-gpu, unix-cpu, unix-gpu, centos-cpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@mseth10 mseth10 requested a review from josephevans February 23, 2022 22:02
@mseth10 mseth10 added pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Feb 23, 2022
Copy link
Contributor

@josephevans josephevans 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!

@RafLit
Copy link
Contributor

RafLit commented Feb 24, 2022

this has been already fixed in #20903

@mseth10 mseth10 closed this Feb 24, 2022
@mseth10 mseth10 deleted the mseth10-fix-website-pipeline branch February 24, 2022 08:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-work-in-progress PR is still work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants