Commit Graph

238 Commits

Author SHA1 Message Date
akhilac1 78caa028b2 #682 APP_PORT not injected into app container
Signed-off-by: akhilac1 <chetlapalle.akhila@gmail.com>
2022-06-04 12:59:48 +05:30
Paul Yuknewicz c0e29ad8c2
Sets APP_PORT in env var of kubernetes yaml and sets default value in… (#683)
* Sets APP_PORT in env var of kubernetes yaml and sets default value in code if not set.  Fixes #682

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

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

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* fixing to point to public container image vs test image

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
2022-06-03 23:04:39 -07:00
Paul Yuknewicz 5471679e6f
Merge pull request #674 from Arashrohani/new-subscriber
Updated makefile and Dockerfile to fix the issue related to dotnet docker build
2022-06-01 00:39:03 -07:00
AkhilaC f76035198b
Fix quickstarts to use app-port (#675)
Signed-off-by: akhilac1 <chetlapalle.akhila@gmail.com>
2022-05-31 16:05:37 -07:00
Arash Rohani 67002ec9b9 Updated makefile and Dockerfile to fix the issue related to dotnet docker build for arm
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
2022-05-27 09:26:37 -07:00
Paul Yuknewicz 74bd70789c
Revert "Updated the docker file to use linux arm image" 2022-05-26 23:50:28 -07:00
Arash Rohani 75bc8735ae Updated the docker file to use linux arm image
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
2022-05-25 20:21:54 -07:00
Arash Rohani a7a497d131 Updated the README regarding the C# changes and fixed a typo.
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Arash Rohani 53aeab771b Removed the whole if condition section.
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Arash Rohani 363bcffe5b Removed Swagger support and Updated the missing steps in README.
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Arash Rohani e2012ef513 Updated makefile and readme.me to fix the build issue.
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Arash Rohani 5c6fc9b029 Upgraded to ASP.NET Core 6 Minimal API
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Arash Rohani 45c7bd5d88 Updated the README.md, Added a new yaml file and updated the appsettings.json to use the new port.
Signed-off-by: Arash Rohani <arash.rohani@gmail.com>
2022-05-24 10:23:25 -07:00
Tony Surma 71e5489c6d
add folder paths to launch for split projects (#493)
Signed-off-by: tonysurma <5067595+tonysurma@users.noreply.github.com>

Co-authored-by: tonysurma <5067595+tonysurma@users.noreply.github.com>
2022-05-23 15:05:50 -07:00
amulyavarote cf3054f04e
Added secrets management examples with the workflow (#653)
* Added secrets management examples with the workflow

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Fixed syntax error for validation

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

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

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changed hello-kubernetes readme

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changes based on the review comments

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Resolved merge conflicts

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Removed k8s installation from secrets mgnt

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Removed dapr k8s installation from secrets mgnt

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>
2022-04-29 14:06:00 -07:00
amulyavarote 27d33638f1
Modified k8s readme to show API logs (#660)
* Modified k8s readme to show API logs

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changed kind config

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverted kind file changes

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverting the workflow changes

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changing docker version

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changed readme sleeps for kubernetes rollout

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changed docker install version

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverted the docker install version

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverted all workflow changes

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changes based on the review comments

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Update tutorials/hello-kubernetes/README.md

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverted the change

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Fixing Python App API logging

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reduced memory in yaml

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Added kind

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Add cluster info - commented

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Added cluster info

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Added cluster info - commented

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Changed MacOS version

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Reverted yaml file changes

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Removed k8s from quickstarts

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Removed dapr kubernetes for quickstarts

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

Co-authored-by: Mark Fussell <markfussell@gmail.com>
2022-04-27 13:02:07 -07:00
Alessandro (Ale) Segala 3f54324acb
Updated hello-world/-kuberentes Node.js apps (#642)
* 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 <43508+ItalyPaleAle@users.noreply.github.com>

* 💄
2022-04-27 12:16:42 -07:00
greenie-msft 55a3d514c0
Fix vulnerabilities in distributed calc (#629)
* Fix dependency vulnerabilities in distributed calc

Signed-off-by: Nick Greenfield <nigreenf@microsoft.com>

* Fix vulnerabilities in distributed calc dependencies

Signed-off-by: Nick Greenfield <nigreenf@microsoft.com>
2022-04-27 12:16:09 -07:00
Doug Davis ca899f73af
Fix some paths since there's a "tutorial" dir now (#631)
Found by Nir Mashkowski

Signed-off-by: Doug Davis <dug@microsoft.com>
2022-04-26 21:38:08 -07:00
Patrick Huber 5b97bf5427
Update README.md (#634)
* Update README.md

fixes path in documentation

* Update README.md

fixes folder

* Update README.md

fixes folder
2022-04-26 21:37:13 -07:00
Doug Davis c21a8f58d5
Reorder quickstarts so the processor is run before the publisher (#637)
Signed-off-by: Doug Davis <dug@microsoft.com>
2022-04-07 16:29:15 -07:00
Doug Davis 2303dab82e
Put reader in 'node' dir (#595)
W/o this change the reader might not realize they need to cd into the
'node' dir to see the app.js file that we talk about next.

Also removed some extra whitespace at the end of some lines.

Signed-off-by: Doug Davis <dug@microsoft.com>
2022-03-23 17:41:27 -07:00
Paul Yuknewicz 107cd31179
Adding "slowness" back to Observability tutorial (#620)
* moving devcontainer to root to enable

* Fix for tutorials to point to new dapr-cli-link

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed additional broken links found by MMD `make validate` in tutorials.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing readme to call python3 instead of python per best practices

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed zipkin api bug failing tests.  Duration with current multiply code will never be slow enough to return traces results!

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Putting "slowness" back in multiplier app by pulling the -slow container from GHCR

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Adding ignore link for fixed zipkin:19411 on K8s link

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* fixing test to match observed k8s behavior with multiplyapp apply

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
2022-03-22 17:19:37 -07:00
Paul Yuknewicz 009a9adae8
Fixing regressions to broken tests (new link) and dev container (#617)
* moving devcontainer to root to enable

* Fix for tutorials to point to new dapr-cli-link

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed additional broken links found by MMD `make validate` in tutorials.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing readme to call python3 instead of python per best practices

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed zipkin api bug failing tests.  Duration with current multiply code will never be slow enough to return traces results!

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
2022-03-21 16:28:16 -07:00
Artur Souza 21138fe564 Merge remote-tracking branch 'upstream/master' into upstream_quickstarts
Signed-off-by: Artur Souza <artursouza.ms@outlook.com>
2022-03-18 14:36:59 -07:00
amulyavarote fcd38b7324
Merge branch 'feature/new_quickstarts' into feature/new_quickstarts 2022-03-18 12:29:20 -07:00
Amulya Varote 8b3bc3d550 Renamed workflows and changes licenses
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:10:54 -07:00
amulyavarote a84baa428e Added validations for service invocation (#607)
* Added validations for service invocation

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>

* Fixed java readme file

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>

* Removing csharp validation

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added license to yaml files

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changes based on the previous PR merge

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* test fix

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed validation

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* tests fix trial

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* test fixes for python

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added seperate workflows to quickstarts

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed failed docker link

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:10:53 -07:00
greenie-msft eb036f4afb Merge updates from master into feature/new_quickstarts (#573)
* Bump follow-redirects in /distributed-calculator/react-calculator/client (#530)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump follow-redirects in /pub-sub/react-form/client (#531)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>

* Refresh pub-sub quickstart (tutorial) fixing routes and stale dependency (#539)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Paulyuk/fixpubsuberror (#547)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Hotfix to test, given precision sensitivity on test output number

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Improved MM test fix for quickstarts (#548)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Hotfix to test, given precision sensitivity on test output number

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing all remaining decimal precision issues with MM tests in readmes, including observability which now fails

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Fix curl sample to work on macOS (#543)

Signed-off-by: Marcin Jahn <marcinj10@gmail.com>

* Fix sample path in readme (#555)

Signed-off-by: Xin Shi <xshi@microsoft.com>

Co-authored-by: Mark Fussell <mfussell@microsoft.com>

* Remove PR check for docker login (#565)

* remove PR check for docker login

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* remove PR trigger for the workflow to fix docker login issue

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

* Test validation cleanup (#569)

* Test validation cleanup

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* add one response message in each of the validation steps

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
Co-authored-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Marcin Jahn <marcinj10@gmail.com>
Co-authored-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: tanvigour <60332928+tanvigour@users.noreply.github.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:10:53 -07:00
amulyavarote 711c1fc026 Added workflow dispatch to trigger the validation (#546)
* Added workflow dispatch

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed from build file

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation test

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation tests

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation tests

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:10:53 -07:00
amulyavarote 782eab7271 Testing GitHub actions for new quickstarts directory structure (#538)
* Modified github actions and the file paths

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed file paths in github actions

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed file paths in github actions in build yaml

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed branch name for external link validation

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added validation file

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Modified paths for the validation to run

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Modified paths for the validation to run

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed broken paths in all readmes

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed unnecessary files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added back removed files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added missing files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changes based on the review comments

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added removed files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed branch name to test

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added master back

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:10:52 -07:00
Nick Greenfield 3bd0c1e089 Move existing quickstarts into tutorials directory.
Signed-off-by: Nick Greenfield <nigreenf@microsoft.com>
Signed-off-by: Artur Souza <artursouza.ms@outlook.com>
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 12:09:56 -07:00
amulyavarote 066424db45
Renamed workflows and changes licenses (#609)
Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>

Co-authored-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 11:31:24 -07:00
amulyavarote b3b1e2d6a5
Added validations for service invocation (#607)
* Added validations for service invocation

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>

* Fixed java readme file

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>

* Removing csharp validation

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added license to yaml files

Signed-off-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changes based on the previous PR merge

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* test fix

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed validation

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* tests fix trial

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* test fixes for python

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added seperate workflows to quickstarts

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed failed docker link

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@Amulyas-MacBook-Pro.local>
2022-03-18 10:45:04 -07:00
greenie-msft 31583775fb
Merge updates from master into feature/new_quickstarts (#573)
* Bump follow-redirects in /distributed-calculator/react-calculator/client (#530)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump follow-redirects in /pub-sub/react-form/client (#531)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>

* Refresh pub-sub quickstart (tutorial) fixing routes and stale dependency (#539)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Paulyuk/fixpubsuberror (#547)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Hotfix to test, given precision sensitivity on test output number

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Improved MM test fix for quickstarts (#548)

* Fixing error caused by route to * conflicing with /dapr/subscribe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrating from `requests` to `axios` to improve supportability

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changed from " to ' matching existing styles for Javascript

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Changing `axios` from import to require('axios'), matching require('express')

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Consistent use of Async enabled http code

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixed react * route bug that conflicts with /dapr/subscribe route

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Reducing log levels introduced by migration.  added debug flag.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating MM test to match improved precision of Express server.js with Axios.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* updating console.log from console.debug (minor) for consistency.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updated vars to const, and then updated README to match.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Async posts needed await.  Readme's tweaked to match updates.

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Update pub-sub/README.md

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Async should pair with await

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing comment

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Removing commented out pipe

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in.  Updated dependencies

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Hotfix to test, given precision sensitivity on test output number

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

* Fixing all remaining decimal precision issues with MM tests in readmes, including observability which now fails

Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>

* Fix curl sample to work on macOS (#543)

Signed-off-by: Marcin Jahn <marcinj10@gmail.com>

* Fix sample path in readme (#555)

Signed-off-by: Xin Shi <xshi@microsoft.com>

Co-authored-by: Mark Fussell <mfussell@microsoft.com>

* Remove PR check for docker login (#565)

* remove PR check for docker login

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* remove PR trigger for the workflow to fix docker login issue

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

* Test validation cleanup (#569)

* Test validation cleanup

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* add one response message in each of the validation steps

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
Co-authored-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Marcin Jahn <marcinj10@gmail.com>
Co-authored-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: tanvigour <60332928+tanvigour@users.noreply.github.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
2022-03-03 16:16:46 -08:00
amulyavarote 90a125f6a4
Added workflow dispatch to trigger the validation (#546)
* Added workflow dispatch

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed from build file

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation test

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation tests

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed validation tests

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>
2022-02-27 13:08:48 -08:00
amulyavarote 3ccd3d3106
Testing GitHub actions for new quickstarts directory structure (#538)
* Modified github actions and the file paths

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed file paths in github actions

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed file paths in github actions in build yaml

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed branch name for external link validation

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added validation file

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Modified paths for the validation to run

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Modified paths for the validation to run

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Fixed broken paths in all readmes

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Removed unnecessary files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added back removed files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added missing files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changes based on the review comments

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added removed files

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Changed branch name to test

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

* Added master back

Signed-off-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>

Co-authored-by: Amulya Varote <amulyavarote@QTM-SWATHIKIL-1.redmond.corp.microsoft.com>
2022-02-22 10:49:20 -08:00
Nick Greenfield 9b9837d75a Move existing quickstarts into tutorials directory.
Signed-off-by: Nick Greenfield <nigreenf@microsoft.com>
Signed-off-by: Artur Souza <artursouza.ms@outlook.com>
2022-02-14 15:49:04 -08:00