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.

Sign up for GitHub

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

Jump to bottom

Update eShop to use Recipes #590

Merged
merged 31 commits into from
Nov 9, 2023
Merged

Update eShop to use Recipes #590

merged 31 commits into from
Nov 9, 2023

Conversation

willdavsmith
Copy link
Contributor

@willdavsmith willdavsmith commented Oct 11, 2023 •
edited
Loading

  • Update eshop to use recipes
  • update test Github workflow

Ex. Successful run: https://github.com/radius-project/samples/actions/runs/6511360118?pr=590

@willdavsmith willdavsmith requested a review from a team as a code owner October 11, 2023 16:27
@willdavsmith willdavsmith changed the title [WIP/ Do not review] Use recipes in eShop sample Use recipes in eShop sample Oct 12, 2023
@willdavsmith willdavsmith mentioned this pull request Nov 8, 2023
Closed
@AaronCrawfis AaronCrawfis changed the title Use recipes in eShop sample Update eShop to use Recipes Nov 9, 2023
@AaronCrawfis
Update env names
10dccb6 
Signed-off-by: Aaron Crawfis <[email protected]>
AaronCrawfis
AaronCrawfis reviewed Nov 9, 2023
View reviewed changes
samples/eshop/eshop.bicep
param imageRegistry string = 'ghcr.io/radius-project/samples/eshop'

@description('Container image tag to use for eshop images. Defaults to "latest".')
param imageTag string = 'latest'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@willdavsmith had to update this back to latest since we didn't have a linux-dotnet7 tag in our package list

@AaronCrawfis
Fix namespaces
dbdadf5 
Signed-off-by: Aaron Crawfis <[email protected]>
ytimocin
ytimocin reviewed Nov 9, 2023
View reviewed changes
samples/eshop/environments/aws.bicep
Comment on lines +44 to +45
// Use containerized RabbitMQ instead of Amazon SQS
// https://github.com/radius-project/bicep-types-aws/blob/main/docs/reference/limitations.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a TODO?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not directly. This is more of a comment of why we're using the local-dev Recipe instead of the Azure service, which is non-idempotent and not supported by Radius. Once we add non-idempotent service support we could come back and update this later.

samples/eshop/infra/infra.bicep
Comment on lines +109 to +113
@description('The name of the RabbitMQ portable resource')
output rabbitmq string = rabbitmq.name

@description('The name of the Service Bus portable resource')
output servicebus string = servicebus.name
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could these cause errors because of conditional resource rendering? For example, rabbitmq is only created if AZURESERVICEBUSENABLED is false.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't hit any errors when testing this. I believe this is ok because the name is a deploy time constant in Bicep/UCP so it will always be the same in this file and isn't dependent on the actual resource being created, like a property such as connection string or hostname would be

samples/eshop/services/basket.bicep
Comment on lines -64 to +54
OrchestratorType: ORCHESTRATOR_TYPE
ORCHESTRATOR_TYPE: 'K8S'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did we need to change this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We used to have this be a parameter that could be set to only the value 'K8S'. Since we don't support anything else we just hard coded it. Goal is to simplify the parameter list

@AaronCrawfis
Fix recipe
e1af6b2 
Signed-off-by: Aaron Crawfis <[email protected]>
@willdavsmith willdavsmith merged commit 46421a0 into edge Nov 9, 2023
13 of 14 checks passed
@willdavsmith willdavsmith deleted the willdavsmith/3-eshop-recipes branch November 9, 2023 21:24
willtsai pushed a commit to willtsai/samples-radius that referenced this pull request Dec 13, 2023
@willdavsmith @AaronCrawfis @willtsai
Update eShop to use Recipes (radius-project#590)
352a2a6 
* Replace eShop with eShop on recipes

* PR

* Fix

* Test

* PR

* oops

* Updates

* PR

* PR

* PR

* PR comments

* PR comments

* fixing

* again

* fix

* rewind

* PR

* PR

* Simplify env registration

Signed-off-by: Aaron Crawfis <[email protected]>

* Make registry a parameter

Signed-off-by: Aaron Crawfis <[email protected]>

* Update test

Signed-off-by: Aaron Crawfis <[email protected]>

* Update namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix tag

Signed-off-by: Aaron Crawfis <[email protected]>

* Update env names

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix recipe

Signed-off-by: Aaron Crawfis <[email protected]>

---------

Signed-off-by: Aaron Crawfis <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
jonvsm pushed a commit that referenced this pull request Dec 13, 2023
@willtsai @willdavsmith
@AaronCrawfis @dependabot @ASHIQUEMD
Upmerge v0.27 to v0.28 (#903)
175425b 
* Add edge rad CLI pull to samples tests (#737)

* Add install-radius script

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

* chmod

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

* test

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

* test

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

* test

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

* test

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

* test

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

* test

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

* test

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

* PR

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

* PR

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

* PR

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

* PR

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

* PR

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

* PR

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

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* dapr tutorial port and routes fixes

Signed-off-by: Will Tsai <[email protected]>

* Revert "Update container build to only push on merges (#726)"

This reverts commit 54b3d4e.

Signed-off-by: Will Tsai <[email protected]>

* Revert "Revert "Update container build to only push on merges (#726)""

This reverts commit 7cfd4ad.

Signed-off-by: Will Tsai <[email protected]>

* Revert "dapr tutorial port and routes fixes"

This reverts commit 7f6f525.

Signed-off-by: Will Tsai <[email protected]>

* dapr tutorial port and routes fixes (#748)

Signed-off-by: Will Tsai <[email protected]>

* Update condition

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Update rad-bicep pull

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Update eShop to use Recipes (#590)

* Replace eShop with eShop on recipes

* PR

* Fix

* Test

* PR

* oops

* Updates

* PR

* PR

* PR

* PR comments

* PR comments

* fixing

* again

* fix

* rewind

* PR

* PR

* Simplify env registration

Signed-off-by: Aaron Crawfis <[email protected]>

* Make registry a parameter

Signed-off-by: Aaron Crawfis <[email protected]>

* Update test

Signed-off-by: Aaron Crawfis <[email protected]>

* Update namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix tag

Signed-off-by: Aaron Crawfis <[email protected]>

* Update env names

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix recipe

Signed-off-by: Aaron Crawfis <[email protected]>

---------

Signed-off-by: Aaron Crawfis <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Fix Dapr sample in CI (#817)

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* make sure it's not flaky

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

* remove debug

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

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Force delete Radius application, EKS Cluster, and AWS resources on failed workflow run (#843)

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Remove customer label

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Add issue labels (#858)

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Bump @adobe/css-tools from 4.3.1 to 4.3.2 in /samples/demo/client (#886)

Bumps [@adobe/css-tools](https://github.com/adobe/css-tools) from 4.3.1 to 4.3.2.
- [Changelog](https://github.com/adobe/css-tools/blob/main/History.md)
- [Commits](https://github.com/adobe/css-tools/commits)

---
updated-dependencies:
- dependency-name: "@adobe/css-tools"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Will Tsai <[email protected]>

* Fix Dapr microservices sample (#654) (#890)

* Adding application name for dapr app

* Adding application name for dapr app

* Update samples/dapr/.rad/rad.yaml

---------

Signed-off-by: Will <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Co-authored-by: MD Ashique <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Will <[email protected]>
Co-authored-by: Will Smith <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: MD Ashique <[email protected]>
willdavsmith added a commit that referenced this pull request Apr 22, 2024
@willdavsmith @AaronCrawfis
@dependabot @jkotalik @willtsai @ASHIQUEMD @lakshmimsft @kachawla @ytimocin @Reshrahim
Upmerge release branches into edge (#1212)
2717f4e 
* Fix resource type

* Fix Dapr mode

* Bump express from 4.17.1 to 4.17.3 in /quickstarts/dapr/nodeapp (#108)

Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.17.3.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.17.1...4.17.3)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump loader-utils from 2.0.3 to 2.0.4 in /demo/client (#111)

Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.3 to 2.0.4.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.4/CHANGELOG.md)
- [Commits](webpack/loader-utils@v2.0.3...v2.0.4)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump minimatch from 3.0.4 to 3.1.2 in /tutorial (#114)

Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.0.4 to 3.1.2.
- [Release notes](https://github.com/isaacs/minimatch/releases)
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.0.4...v3.1.2)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump decode-uri-component from 0.2.0 to 0.2.2 in /tutorial (#113)

Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2.
- [Release notes](https://github.com/SamVerschueren/decode-uri-component/releases)
- [Commits](SamVerschueren/decode-uri-component@v0.2.0...v0.2.2)

---
updated-dependencies:
- dependency-name: decode-uri-component
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump minimatch in /reference-apps/container-app-store/node-service (#112)

Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.0.4 to 3.1.2.
- [Release notes](https://github.com/isaacs/minimatch/releases)
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.0.4...v3.1.2)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump qs and express in /reference-apps/container-app-store/node-service (#110)

Bumps [qs](https://github.com/ljharb/qs) to 6.11.0 and updates ancestor dependency [express](https://github.com/expressjs/express). These dependencies need to be updated together.


Updates `qs` from 6.5.2 to 6.11.0
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.5.2...v6.11.0)

Updates `express` from 4.16.4 to 4.18.2
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.16.4...4.18.2)

---
updated-dependencies:
- dependency-name: qs
  dependency-type: indirect
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Bump express from 4.17.1 to 4.17.3 in /tutorial (#109)

Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.17.3.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.17.1...4.17.3)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

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

* Making tutorial match docs version (#116)

* Continue on error for container deletion

* Update issues.yaml

* Change ADO_wit

* Update version number

* Remove container app store

* Add eshop instructions

* Revert instructions

* Change Bicep installation step

* Update issues.yaml

* Update issues.yaml

* Update issues.yaml

* Update test.yaml

* Add check for node creation (#240)

* Add check for node creation

* Correct error message

* Add contribution guide

* Add Issue templates

* Fix link

* Fix casing

* change mentions of <Project Radius> to just <Radius> (#263)

Signed-off-by: Will Tsai <[email protected]>

* edge to v0.22 upmerge to pick up  changes (#388)

Signed-off-by: Will Tsai <[email protected]>

* add background info for dapr quickstart (#381)

Signed-off-by: Will Tsai <[email protected]>

* Switch to sql edge (#398)

* Update Dapr UI port (#416)

* Change port to 8080

* Update container port

* Update configuration

* Add error message

* Update demo.app.spec.ts (#546)

* Add samples release automation (#499) (#554)

* Bump word-wrap from 1.2.3 to 1.2.5 in /demo/client (#431)

Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.5.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.5)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Aaron Crawfis <[email protected]>

* Bump golang.org/x/net in /reference-apps/container-app-store/go-service (#593)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.0.0-20210929193557-e81a3d93ecf6 to 0.17.0.
- [Commits](https://github.com/golang/net/commits/v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

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

* Add issue labels (#858)

Signed-off-by: Aaron Crawfis <[email protected]>

* Remove customer label

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix failing CI in 0.27 branch  (#881)

* Fix Dapr sample in CI (#817)

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* make sure it's not flaky

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

* remove debug

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

---------

Signed-off-by: willdavsmith <[email protected]>
(cherry picked from commit 5f6bd44)

* Force delete Radius application, EKS Cluster, and AWS resources on failed workflow run (#843)

Signed-off-by: willdavsmith <[email protected]>
(cherry picked from commit 4855d47)

* Bump @adobe/css-tools from 4.3.1 to 4.3.2 in /samples/demo/client (#886)

Bumps [@adobe/css-tools](https://github.com/adobe/css-tools) from 4.3.1 to 4.3.2.
- [Changelog](https://github.com/adobe/css-tools/blob/main/History.md)
- [Commits](https://github.com/adobe/css-tools/commits)

---
updated-dependencies:
- dependency-name: "@adobe/css-tools"
  dependency-type: indirect
...

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

* Fix Dapr microservices sample (#654)

* Adding application name for dapr app

* Adding application name for dapr app

* Update samples/dapr/.rad/rad.yaml

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

---------

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

* Fix Dapr microservices sample (#654) (#890)

* Adding application name for dapr app

* Adding application name for dapr app

* Update samples/dapr/.rad/rad.yaml



---------

Signed-off-by: Will <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Co-authored-by: MD Ashique <[email protected]>

* Upmerge v0.27 to v0.28 (#903)

* Add edge rad CLI pull to samples tests (#737)

* Add install-radius script

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

* chmod

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

* test

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

* test

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

* test

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

* test

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

* test

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

* test

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

* test

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

* PR

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

* PR

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

* PR

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

* PR

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

* PR

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

* PR

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

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* dapr tutorial port and routes fixes

Signed-off-by: Will Tsai <[email protected]>

* Revert "Update container build to only push on merges (#726)"

This reverts commit 54b3d4e.

Signed-off-by: Will Tsai <[email protected]>

* Revert "Revert "Update container build to only push on merges (#726)""

This reverts commit 7cfd4ad.

Signed-off-by: Will Tsai <[email protected]>

* Revert "dapr tutorial port and routes fixes"

This reverts commit 7f6f525.

Signed-off-by: Will Tsai <[email protected]>

* dapr tutorial port and routes fixes (#748)

Signed-off-by: Will Tsai <[email protected]>

* Update condition

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Update rad-bicep pull

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Update eShop to use Recipes (#590)

* Replace eShop with eShop on recipes

* PR

* Fix

* Test

* PR

* oops

* Updates

* PR

* PR

* PR

* PR comments

* PR comments

* fixing

* again

* fix

* rewind

* PR

* PR

* Simplify env registration

Signed-off-by: Aaron Crawfis <[email protected]>

* Make registry a parameter

Signed-off-by: Aaron Crawfis <[email protected]>

* Update test

Signed-off-by: Aaron Crawfis <[email protected]>

* Update namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix tag

Signed-off-by: Aaron Crawfis <[email protected]>

* Update env names

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix namespaces

Signed-off-by: Aaron Crawfis <[email protected]>

* Fix recipe

Signed-off-by: Aaron Crawfis <[email protected]>

---------

Signed-off-by: Aaron Crawfis <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Fix Dapr sample in CI (#817)

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* Adding debug for pod list

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

* make sure it's not flaky

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

* remove debug

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

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Force delete Radius application, EKS Cluster, and AWS resources on failed workflow run (#843)

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Remove customer label

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Add issue labels (#858)

Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

* Bump @adobe/css-tools from 4.3.1 to 4.3.2 in /samples/demo/client (#886)

Bumps [@adobe/css-tools](https://github.com/adobe/css-tools) from 4.3.1 to 4.3.2.
- [Changelog](https://github.com/adobe/css-tools/blob/main/History.md)
- [Commits](https://github.com/adobe/css-tools/commits)

---
updated-dependencies:
- dependency-name: "@adobe/css-tools"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Will Tsai <[email protected]>

* Fix Dapr microservices sample (#654) (#890)

* Adding application name for dapr app

* Adding application name for dapr app

* Update samples/dapr/.rad/rad.yaml

---------

Signed-off-by: Will <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Co-authored-by: MD Ashique <[email protected]>
Signed-off-by: Will Tsai <[email protected]>

---------

Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Will <[email protected]>
Co-authored-by: Will Smith <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: MD Ashique <[email protected]>

* Update changes in test.yaml workflow into v0.28 (#904)

* Add test-failure label to issues created for samples failures (#899)

Signed-off-by: karishma-chawla <[email protected]>
Co-authored-by: karishma-chawla <[email protected]>

* updating condition to specify scheduled run

---------

Signed-off-by: karishma-chawla <[email protected]>
Signed-off-by: Lakshmi Javadekar <[email protected]>
Co-authored-by: Karishma Chawla <[email protected]>
Co-authored-by: karishma-chawla <[email protected]>

* Fixing AWS RDS Purge workflow (#970)

* Add aws login (#974)

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

* Add GH action to close stale PRs (#977)

* Add GH action to close stale PRs

Signed-off-by: karishma-chawla <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Aaron Crawfis <[email protected]>
Signed-off-by: Karishma Chawla <[email protected]>

* Update radius repo link

Signed-off-by: Karishma Chawla <[email protected]>

---------

Signed-off-by: karishma-chawla <[email protected]>
Signed-off-by: Karishma Chawla <[email protected]>
Co-authored-by: karishma-chawla <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>

* Adding a secton that waits until the app is ready (#980)

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

* Update images for guestbook sample app (#986)

* update images for guestbook sample app

Signed-off-by: Will Tsai <[email protected]>

* update images for guestbook sample app

Signed-off-by: Will Tsai <[email protected]>

* move images to /samples path

Signed-off-by: Will Tsai <[email protected]>

---------

Signed-off-by: Will Tsai <[email protected]>

* Bump webpack-dev-middleware from 5.3.3 to 5.3.4 in /samples/demo/client (#1023)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 5.3.3 to 5.3.4.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v5.3.4/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v5.3.3...v5.3.4)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
...

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

* Bump follow-redirects from 1.15.4 to 1.15.6 in /samples/demo/client (#1014)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.4 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.4...v1.15.6)

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

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

* Bump ip from 2.0.0 to 2.0.1 in /samples/demo (#984)

Bumps [ip](https://github.com/indutny/node-ip) from 2.0.0 to 2.0.1.
- [Commits](indutny/node-ip@v2.0.0...v2.0.1)

---
updated-dependencies:
- dependency-name: ip
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will <[email protected]>

* Bump follow-redirects from 1.15.5 to 1.15.6 in /playwright (#1015)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6)

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

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will <[email protected]>

* Bump express from 4.18.2 to 4.19.2 in /samples/demo/client (#1027)

Bumps [express](https://github.com/expressjs/express) from 4.18.2 to 4.19.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.18.2...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will <[email protected]>

* Bump express from 4.18.2 to 4.19.2 in /samples/demo (#1028)

Bumps [express](https://github.com/expressjs/express) from 4.18.2 to 4.19.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.18.2...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will <[email protected]>

* Bump express from 4.18.2 to 4.19.2 in /samples/dapr/nodeapp (#1026)

Bumps [express](https://github.com/expressjs/express) from 4.18.2 to 4.19.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.18.2...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will <[email protected]>

* Make env configurable (#1046)

Signed-off-by: Reshma Abdul Rahim <[email protected]>

* Adding test AKS workflow to v0.32 (#1052)

* Run functional tests every 2 hours and add another workflow that runs… (#1020)

* Run functional tests every 2 hours and add another workflow that runs the tests on AKS instead of k3d

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

* Triggering workflow

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

---------

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

* Removing the run of the AKS workflow on PRs (#1051)

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

* Update the timeout (30s) for Playwright

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

---------

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

* Removing code

* Removing code

* Removing code

* Use retry and update eshop playwright tests to wait for the catalog to appear (#1213)

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

* Fixing eshop page

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

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Will Tsai <[email protected]>
Signed-off-by: Aaron Crawfis <[email protected]>
Signed-off-by: Will <[email protected]>
Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: karishma-chawla <[email protected]>
Signed-off-by: Lakshmi Javadekar <[email protected]>
Signed-off-by: Karishma Chawla <[email protected]>
Signed-off-by: ytimocin <[email protected]>
Signed-off-by: Reshma Abdul Rahim <[email protected]>
Co-authored-by: Aaron Crawfis <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Justin Kotalik <[email protected]>
Co-authored-by: Will <[email protected]>
Co-authored-by: MD Ashique <[email protected]>
Co-authored-by: Lakshmi Javadekar <[email protected]>
Co-authored-by: Karishma Chawla <[email protected]>
Co-authored-by: karishma-chawla <[email protected]>
Co-authored-by: Yetkin Timocin <[email protected]>
Co-authored-by: Reshma Abdul Rahim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AaronCrawfis AaronCrawfis AaronCrawfis approved these changes

@ytimocin ytimocin ytimocin left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@willdavsmith @ytimocin @AaronCrawfis