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

adapt breaking changes of dapr 1.0.0.rc.1 #883

Merged
merged 2 commits into from
Jan 7, 2021

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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy