Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

thoemmi
Copy link
Contributor

@thoemmi thoemmi commented Jan 7, 2021

Dapr v1.0.0-rc-1 introduced following breaking change:

The following annotations are no longer supported:

  • dapr.io/id
  • dapr.io/port
  • dapr.io/protocol

Instead, use the following:

  • dapr.io/app-id
  • dapr.io/app-port
  • dapr.io/app-protocol

This PR fixes the renamed annotations when deploying with Tye to k8s.

@jkotalik
Copy link
Contributor

jkotalik commented Jan 7, 2021

Looks like a test needs to be updated in this PR. Generate_DaprApplication

@thoemmi
Copy link
Contributor Author

thoemmi commented Jan 7, 2021

@jkotalik I've fixed the failing test, but now the OSX build fails in IngressStaticFilesTest with an internal server error.

@jkotalik
Copy link
Contributor

jkotalik commented Jan 7, 2021

Looks like a flaky test.

@thoemmi
Copy link
Contributor Author

thoemmi commented Jan 7, 2021

Yeah, the second run succeeded 👍

@jkotalik jkotalik merged commit 9381345 into dotnet:master Jan 7, 2021
@thoemmi thoemmi deleted the deploy-with-dapr-1.0.0-rc.2 branch January 7, 2021 23:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants