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

Add bindings test #1

Open
wants to merge 71 commits into
base: master
Choose a base branch
from
Open

Add bindings test #1

wants to merge 71 commits into from

Conversation

greenie-msft
Copy link
Owner

Description

Please explain the changes you've made

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.

tanvigour and others added 30 commits March 29, 2022 11:55
* Changed docker installation version

Signed-off-by: Amulya Varote <[email protected]>

* Added virtual box to the minikube

Signed-off-by: Amulya Varote <[email protected]>

* Reduced memory size

Signed-off-by: Amulya Varote <[email protected]>

* Reduced memory size

Signed-off-by: Amulya Varote <[email protected]>

* Updated docker version in all the workflows

Signed-off-by: Amulya Varote <[email protected]>

* Changed version for tutorials

Signed-off-by: Amulya Varote <[email protected]>

* Reverted version change

Signed-off-by: Amulya Varote <[email protected]>

* Changed mac os version back to 10.15

Signed-off-by: Amulya Varote <[email protected]>

* Reverted macos version back to latest

Signed-off-by: Amulya Varote <[email protected]>
* Update README.md

fixes path in documentation

* Update README.md

fixes folder

* Update README.md

fixes folder
Found by Nir Mashkowski

Signed-off-by: Doug Davis <[email protected]>
* Fix dependency vulnerabilities in distributed calc

Signed-off-by: Nick Greenfield <[email protected]>

* Fix vulnerabilities in distributed calc dependencies

Signed-off-by: Nick Greenfield <[email protected]>
* Updated hello-world/-kuberentes Node.js apps

1. Using async/await rather than promises for better readability
2. Make sure that methods do not return until state is committed

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

* 💄
* Modified k8s readme to show API logs

Signed-off-by: Amulya Varote <[email protected]>

* Changed kind config

Signed-off-by: Amulya Varote <[email protected]>

* Reverted kind file changes

Signed-off-by: Amulya Varote <[email protected]>

* Reverting the workflow changes

Signed-off-by: Amulya Varote <[email protected]>

* Changing docker version

Signed-off-by: Amulya Varote <[email protected]>

* Changed readme sleeps for kubernetes rollout

Signed-off-by: Amulya Varote <[email protected]>

* Changed docker install version

Signed-off-by: Amulya Varote <[email protected]>

* Reverted the docker install version

Signed-off-by: Amulya Varote <[email protected]>

* Reverted all workflow changes

Signed-off-by: Amulya Varote <[email protected]>

* Changes based on the review comments

Signed-off-by: Amulya Varote <[email protected]>

* Update tutorials/hello-kubernetes/README.md

Signed-off-by: Amulya Varote <[email protected]>

* Reverted the change

Signed-off-by: Amulya Varote <[email protected]>

* Fixing Python App API logging

Signed-off-by: Amulya Varote <[email protected]>

* Reduced memory in yaml

Signed-off-by: Amulya Varote <[email protected]>

* Added kind

Signed-off-by: Amulya Varote <[email protected]>

* Add cluster info - commented

Signed-off-by: Amulya Varote <[email protected]>

* Added cluster info

Signed-off-by: Amulya Varote <[email protected]>

* Added cluster info - commented

Signed-off-by: Amulya Varote <[email protected]>

* Changed MacOS version

Signed-off-by: Amulya Varote <[email protected]>

* Reverted yaml file changes

Signed-off-by: Amulya Varote <[email protected]>

* Removed k8s from quickstarts

Signed-off-by: Amulya Varote <[email protected]>

* Removed dapr kubernetes for quickstarts

Signed-off-by: Amulya Varote <[email protected]>

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

Resolve PR comments

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

Update pub_sub/go/sdk/order-processor/app.go

Co-authored-by: Shubham Sharma <[email protected]>
Signed-off-by: akhilac1 <[email protected]>

Co-authored-by: Shubham Sharma <[email protected]>
* Added secrets management examples with the workflow

Signed-off-by: Amulya Varote <[email protected]>

* Fixed syntax error for validation

Signed-off-by: Amulya Varote <[email protected]>

* Increased time of the rollouts - distributed calc eg using k8s

Signed-off-by: Amulya Varote <[email protected]>

* Changed hello-kubernetes readme

Signed-off-by: Amulya Varote <[email protected]>

* Changes based on the review comments

Signed-off-by: Amulya Varote <[email protected]>

* Resolved merge conflicts

Signed-off-by: Amulya Varote <[email protected]>

* Removed k8s installation from secrets mgnt

Signed-off-by: Amulya Varote <[email protected]>

* Removed dapr k8s installation from secrets mgnt

Signed-off-by: Amulya Varote <[email protected]>
Signed-off-by: Amulya Varote <[email protected]>
* Update Program.cs

Use model binding

* Update Program.cs
* Quickstart example for secret store building block

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

* Changes for auto validate through workflow

Signed-off-by: Pravin Pushkar <[email protected]>
dapr#671)

* Java quickstarts auto validate changes

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

* changing action-setup-docker version for macos failures

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

* Changing actions-setup-docker version to v1

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

* Script for installing docker on macos instead of actions-setup-docker

Signed-off-by: Pravin Pushkar <[email protected]>
* feat(pub_sub/go/sdk): Upgrade Dapr Go SDK to 1.4.0

Signed-off-by: Noah Gao <[email protected]>

* feat(pub_sub/go/sdk): move dapr/dapr to v1.7.2

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

* fix(pubsub/go/sdk): Tidy mod

Signed-off-by: Noah Gao <[email protected]>

* fix(pubsub/go/sdk): Go SDK Subscriber Type change to map

Signed-off-by: Noah Gao <[email protected]>
…ngs.json to use the new port.

Signed-off-by: Arash Rohani <[email protected]>
paulyuk and others added 29 commits June 3, 2022 23:04
dapr#683)

* Sets APP_PORT in env var of kubernetes yaml and sets default value in code if not set.  Fixes dapr#682

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

* fixed whitespace in yaml for env var "value: "

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

* fixing to point to public container image vs test image

Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: akhilac1 <[email protected]>
Signed-off-by: Amulya Varote <[email protected]>
Signed-off-by: Pravin Pushkar <[email protected]>
Signed-off-by: Pravin Pushkar <[email protected]>
* pulling a prebuilt image to try overcoming CodeSpace 'out of storage' issue on build.

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

* trying published MCR image for universal

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

* removing http from tag

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

* Taking working mcr image and adding dockerfile customizations

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

* Update Dockerfile

Updating image pull to use a cached image for CodeSpaces
Fix for failing Auto validate Java examples on State management
* Removed the extra space in expected test result

Signed-off-by: Arash Rohani <[email protected]>

* Changed the output match from exact to substring to fix the issue.

Signed-off-by: Arash Rohani <[email protected]>

* Replaced the logger.LogInformation with Console.WriteLine to fix the test issue.

Signed-off-by: Arash Rohani <[email protected]>

* Fixed the whitespace issue and added sleep directive

Signed-off-by: Arash Rohani <[email protected]>
…ent (dapr#687)

* Fix and normalize the Javascript HTTP and SDK styles of State Management

Signed-off-by: Amulya Varote <[email protected]>

* Change SDK to use @dapr/dapr and construct URL from protocol

Signed-off-by: Amulya Varote <[email protected]>

* Split PORT out from HOST

Signed-off-by: Amulya Varote <[email protected]>

* Formatting and remove unecessary toStrings

Signed-off-by: Amulya Varote <[email protected]>

* Add missing break statements

Signed-off-by: Amulya Varote <[email protected]>

* Use CommunicationProtocolEnum and remove orderId

Signed-off-by: Amulya Varote <[email protected]>

* Fixed state management validation

Signed-off-by: Amulya Varote <[email protected]>

Co-authored-by: Amulya Varote <[email protected]>
* Binding quickstart - Cron input PostgreSQL output
Signed-off-by: Nir Mashkowski <[email protected]>

* Updated to properly load from APP_PORT and DAPR_HTTP_PORT, or default value.

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

* Adds Dockerfile and docker compose file to initialize orders table in postgress in an automated way.

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

* Moving database files to db folder and add README

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

* readme updates

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

* Cleaning up bindings file names

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

* Fixing crash caused by moving orders.json.  keeping in root folder.

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

* Tuning names and flake8 lint pass

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

* Complete code and lint review of Python examples

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

* updating comments with `dapr run ..` that works

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

* Lowercase sqldb binding; fixing request formatting and error handling

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

* removed grpc dependency and use flask for routes

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

* Fixed Axios issues with incorrect playload

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

* fixed env vars and var groups

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

* Fixed forced exit, and comments

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

* Go changes for styling and cont. event handling

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

* Adding READMEs and MMD tests for Python

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

* Adding C# README files and MMD tests

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

* Adding Go README files and MMD tests

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

* Adding Javascript README files and MMD

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

* Renamed `app` files in each language to be consistent with idiom.  e.g. program.cs for C#, index.js for Javascript, and app.python for Python.  Per review feedback from @amulyavarote

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

* updating server port to be string

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

* Changing folder structure for consistency. Nesting language source files in `./batch` for http and sdk variations. Fixing up code and readmes to match.

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

* Adding makefiles for validate (MMD test) and docker (inert)

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

* Adjusting MMD with escape chars and sleeps

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

* Fixed make validate (match, timeouts, sleeps) for C# and Go

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

* Updated MMD with precision fixes.  All passing!

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

* change for java maven builds to build clean

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

* WIP on Java

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

* Java SDK is done!

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

* Java signed off!

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

* pretty formatted POM and java so better with merges in future

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

* Update bindings/javascript/sdk/README.md

Accepted

Co-authored-by: greenie-msft <[email protected]>

Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>
Co-authored-by: greenie-msft <[email protected]>
…ent link (dapr#698)

* updating readme for all quickstarts:
- added link to new Bindings quickstart
- tightened up descriptons (remove "Demonstrates repetition", and better explain the role of the building block in the sample)

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

* Secrets is there too, so updating readme

Signed-off-by: Paul Yuknewicz <[email protected]>
Signed-off-by: Nick Greenfield <[email protected]>
greenie-msft pushed a commit that referenced this pull request Mar 31, 2023
Add workflow to deploy to AKS
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.