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 ORAS install to test workflow #1969

Merged
merged 1 commit into from
Dec 30, 2024
Merged

Conversation

willdavsmith
Copy link
Contributor

Samples test runs are failing because they are trying to pull the rad CLI using oras but it doesn't exist. This PR adds a setup-oras step.

Example: #1967

@willdavsmith willdavsmith requested review from a team as code owners December 30, 2024 17:25
Copy link
Contributor

@ytimocin ytimocin left a comment

Choose a reason for hiding this comment

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

Interestingly this doesn't happen in ubuntu-latest-m. There should have been an update to the ubuntu-latest.

@willdavsmith willdavsmith merged commit 9ecc144 into v0.41 Dec 30, 2024
7 of 8 checks passed
@willdavsmith willdavsmith deleted the willdavsmith/add-oras-2 branch December 30, 2024 17:46
@sk593 sk593 mentioned this pull request Jan 8, 2025
vishwahiremat pushed a commit that referenced this pull request Jan 8, 2025
* Update samples for v0.41

* Bump cross-spawn from 7.0.3 to 7.0.6 in /samples/demo (#1890)

Bumps [cross-spawn](https://github.com/moxystudio/node-cross-spawn) from 7.0.3 to 7.0.6.
- [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md)
- [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6)

---
updated-dependencies:
- dependency-name: cross-spawn
  dependency-type: indirect
...

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

* Add ORAS install to test workflow (#1969)

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

* Updating playwright version to fix the missing packages issue (#1980)

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

* Updating Dapr UI Sample to dotnet 6 (#1970)

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

* Bump nanoid from 3.3.4 to 3.3.8 in /samples/demo/client (#1896)

Dependabot couldn't find the original pull request head commit, 2f3d48c.

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

* Bump path-to-regexp and express in /samples/demo/client (#1893)

Bumps [path-to-regexp](https://github.com/pillarjs/path-to-regexp) and [express](https://github.com/expressjs/express). These dependencies needed to be updated together.

Updates `path-to-regexp` from 0.1.10 to 0.1.12
- [Release notes](https://github.com/pillarjs/path-to-regexp/releases)
- [Changelog](https://github.com/pillarjs/path-to-regexp/blob/master/History.md)
- [Commits](pillarjs/path-to-regexp@v0.1.10...v0.1.12)

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

---
updated-dependencies:
- dependency-name: path-to-regexp
  dependency-type: indirect
- 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>

* Remove AWS CLI install from AWS workflows (#1967)

* Remove AWS CLI install

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

* Add workflow_dispatch to AWS runs

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

---------

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

* Upmerge to edge

Signed-off-by: Radius CI Bot <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: willdavsmith <[email protected]>
Signed-off-by: ytimocin <[email protected]>
Signed-off-by: Radius CI Bot <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will Smith <[email protected]>
Co-authored-by: Yetkin Timocin <[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