Skip to content

Conversation

@Yuqing-cat
Copy link
Collaborator

Signed-off-by: Yuqing Wei [email protected]

Description

Resolves #870, #888

Also add a log to let user know about direct purview link failed.

How was this PR tested?

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

@Yuqing-cat Yuqing-cat added the safe to test Tag to execute build pipeline for a PR from forked repo label Dec 1, 2022
@xiaoyongzhu xiaoyongzhu merged commit bb36583 into feathr-ai:main Dec 2, 2022
jaymo001 pushed a commit that referenced this pull request Dec 7, 2022
* update registry test, modify log

Signed-off-by: Yuqing Wei <[email protected]>

* fix comment

Signed-off-by: Yuqing Wei <[email protected]>

Signed-off-by: Yuqing Wei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Tag to execute build pipeline for a PR from forked repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FR] Add 404, 409 / 412 Exception test in registry test

3 participants