Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

setting version for kafka #939

Merged
merged 2 commits into from
Sep 27, 2023
Merged

Conversation

pravinpushkar
Copy link
Contributor

@pravinpushkar pravinpushkar commented Sep 27, 2023

Description

Setting Kafka to earlier version.

Issue reference

We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • The quickstart code compiles correctly
  • You've tested new builds of the quickstart if you changed quickstart code
  • You've updated the quickstart's README if necessary
  • If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

@pravinpushkar pravinpushkar marked this pull request as ready for review September 27, 2023 11:34
@pravinpushkar pravinpushkar changed the title [Draft] fixing version for kafka fixing version for kafka Sep 27, 2023
@pravinpushkar pravinpushkar changed the title fixing version for kafka setting version for kafka Sep 27, 2023
@pravinpushkar pravinpushkar changed the base branch from master to release-1.12 September 27, 2023 16:37
@yaron2 yaron2 merged commit ab41de2 into dapr:release-1.12 Sep 27, 2023
7 checks passed
@yaron2
Copy link
Member

yaron2 commented Sep 27, 2023

@pravinpushkar why can't we use latest kafka?

@pravinpushkar
Copy link
Contributor Author

@pravinpushkar why can't we use latest kafka?

@yaron2 Not completely sure as of now, but looks like we need to do some change in our kafka setup steps in README. The latest kafka does not comes with zookeeper by default and also it has SASL auth enabled by default. So, probably something related to this as our app sidecar could not connect with Kafka saying "client has run out of available brokers".

msfussell added a commit that referenced this pull request Oct 13, 2023
* update dapr runtime to use v1.12.0-rc.1

Signed-off-by: Cassandra Coyle <[email protected]>

* update to latest rc for runtime and cli

Signed-off-by: Cassandra Coyle <[email protected]>

* try sleeping

Signed-off-by: Cassandra Coyle <[email protected]>

* pin redis version

Signed-off-by: Cassandra Coyle <[email protected]>

* rm sleep

Signed-off-by: Cassandra Coyle <[email protected]>

* rm deprecated unsubscribe call

Signed-off-by: Cassandra Coyle <[email protected]>

* Update configuration/go/sdk/order-processor/app.go

Co-authored-by: Josh van Leeuwen <[email protected]>
Signed-off-by: Cassie Coyle <[email protected]>

* Statemanagement multiapp run

Signed-off-by: rabollin <[email protected]>

* message sequence

Signed-off-by: rabollin <[email protected]>

* message sequence

Signed-off-by: rabollin <[email protected]>

* Multi-run support for workflow

Signed-off-by: Paul Yuknewicz <[email protected]>

* Removing component folder from run file to match readme

Signed-off-by: Paul Yuknewicz <[email protected]>

* aligning state store names on `statestore`

Signed-off-by: Paul Yuknewicz <[email protected]>

* Adding multi-app run for python and C#

Signed-off-by: Paul Yuknewicz <[email protected]>

* pr comments

Signed-off-by: rabollin <[email protected]>

* pr comments

Signed-off-by: rabollin <[email protected]>

* sleep value increase

Signed-off-by: rabollin <[email protected]>

* sleep seconds increase

Signed-off-by: rabollin <[email protected]>

* Result correction

Signed-off-by: rabollin <[email protected]>

* Result correction

Signed-off-by: rabollin <[email protected]>

* java sdk dapr run output format fixing

Signed-off-by: rabollin <[email protected]>

* Syncing Release 1.12 with master and multi-run changes (#936)

* Add holopin.yml config (#923)

Signed-off-by: Marc Duiker <[email protected]>

* Updating Readme's to default to `dapr run -f .` (#874)

* Updating Invoke readme in C# to favor dapr run -f

Signed-off-by: Paul Yuknewicz <[email protected]>

* Adding back single app run steps and tests, but to the end.

Signed-off-by: Paul Yuknewicz <[email protected]>

* Update service_invocation/csharp/http/README.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>

* Updated and hardened test

Signed-off-by: Paul Yuknewicz <[email protected]>

* Invoke * and partial PubSub with multi-run files & readmes & tests

Signed-off-by: Paul Yuknewicz <[email protected]>

* Multi run done for Pubsub C#, Python and Node (http, sdk)

Signed-off-by: Paul Yuknewicz <[email protected]>

* Added Pubsub support for Go, Java

Signed-off-by: Paul Yuknewicz <[email protected]>

* updating readmes with correct appid

Signed-off-by: Paul Yuknewicz <[email protected]>

* Updating tests for PubSub and Service invoke to Paas with multiapp run

Signed-off-by: Paul Yuknewicz <[email protected]>

* More fine tuning of pubsub tests (timeouts, step ends)

Signed-off-by: Paul Yuknewicz <[email protected]>

* Fixing Python FastAPI and C# Pubsub tests

Signed-off-by: Paul Yuknewicz <[email protected]>

---------

Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>

* Patching tests to pass (C# timing, Python uvicorn) (#933)

Signed-off-by: Paul Yuknewicz <[email protected]>

---------

Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Marc Duiker <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>

* Fixing MMD tests with tweaks

Signed-off-by: Paul Yuknewicz <[email protected]>

* setting version for kafka (#939)

* fixing version for kafka

Signed-off-by: Pravin Pushkar <[email protected]>

* validate data with some initial values instead of larger

Signed-off-by: Pravin Pushkar <[email protected]>

---------

Signed-off-by: Pravin Pushkar <[email protected]>

* Updating test env to Dapr runtime 1.12 RC4 and CLI RC2

Signed-off-by: Paul Yuknewicz <[email protected]>

* Update README.md

Fixing typo.  should be singular

Signed-off-by: Paul Yuknewicz <[email protected]>

* Update README.md

Signed-off-by: Paul Yuknewicz <[email protected]>

* add java workflow quickstart

Signed-off-by: Sky Ao <[email protected]>

* Updating runtime for tests to RC5

Signed-off-by: Paul Yuknewicz <[email protected]>

* Fixes port 6003 conflict between Go Pub-sub and Go bindings.  Timing issue

Signed-off-by: Paul Yuknewicz <[email protected]>

* add hello-kubernetes multi-app run k8s template

Signed-off-by: Mukundan Sundararajan <[email protected]>

* address review comments.

Signed-off-by: Mukundan Sundararajan <[email protected]>

* update cli to rc.4 (#943)

* update cli to rc.3

Signed-off-by: Cassandra Coyle <[email protected]>

* update cli to rc.4

Signed-off-by: Cassandra Coyle <[email protected]>

---------

Signed-off-by: Cassandra Coyle <[email protected]>
Signed-off-by: Cassie Coyle <[email protected]>

* Update CLI to rc4

Signed-off-by: Paul Yuknewicz <[email protected]>

* Updating .NET SDK to 1.12.*

Signed-off-by: Paul Yuknewicz <[email protected]>

* Dapr Golang SDK update to v1.9.0-rc-1

Signed-off-by: Paul Yuknewicz <[email protected]>

* Javascript SDK to 3.1.2

Signed-off-by: Paul Yuknewicz <[email protected]>

* Release 1.12 (#945)

* Dapr Golang SDK update to v1.9.0-rc-1

Signed-off-by: Paul Yuknewicz <[email protected]>

* Javascript SDK to 3.1.2

Signed-off-by: Paul Yuknewicz <[email protected]>

---------

Signed-off-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>

* Java SDK update to <version>1.10.0-rc-1</version>

Signed-off-by: Paul Yuknewicz <[email protected]>

* Updated Java SDK to RC, and donated multi-app-run support too :)

Signed-off-by: Paul Yuknewicz <[email protected]>

* Updating tests to use dapr/dapr 1.12.0-rc.6

Signed-off-by: Paul Yuknewicz <[email protected]>

* sdk version changes

Signed-off-by: rabollin <[email protected]>

* CLI and Runtime version changes

Signed-off-by: rabollin <[email protected]>

* go.sum refresh

Signed-off-by: rabollin <[email protected]>

* Updating with js-sdk 3.2.0

Signed-off-by: Paul Yuknewicz <[email protected]>

---------

Signed-off-by: Cassandra Coyle <[email protected]>
Signed-off-by: Cassie Coyle <[email protected]>
Signed-off-by: rabollin <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Marc Duiker <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Pravin Pushkar <[email protected]>
Signed-off-by: Sky Ao <[email protected]>
Signed-off-by: Mukundan Sundararajan <[email protected]>
Co-authored-by: Cassandra Coyle <[email protected]>
Co-authored-by: Cassie Coyle <[email protected]>
Co-authored-by: Josh van Leeuwen <[email protected]>
Co-authored-by: rabollin <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Marc Duiker <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>
Co-authored-by: Pravin Pushkar <[email protected]>
Co-authored-by: Sky Ao <[email protected]>
Co-authored-by: Mukundan Sundararajan <[email protected]>
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.

2 participants