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 support for synchronous mode in rez-pkg-cache #1834

Closed

Conversation

Pixel-Minions
Copy link
Contributor

Add support for synchronous and asynchronous package caching modes via argparse in pkg-cache.py

@Pixel-Minions Pixel-Minions requested a review from a team as a code owner September 26, 2024 18:57
Copy link

codecov bot commented Sep 26, 2024

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 58.51%. Comparing base (8476696) to head (29bf0b5).
Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
src/rez/cli/pkg-cache.py 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1834      +/-   ##
==========================================
- Coverage   58.52%   58.51%   -0.01%     
==========================================
  Files         126      126              
  Lines       17207    17211       +4     
  Branches     3519     3520       +1     
==========================================
+ Hits        10070    10071       +1     
- Misses       6468     6471       +3     
  Partials      669      669              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Pixel-Minions and others added 26 commits September 26, 2024 17:26
…1640)

Signed-off-by: Pavel Vergeev <vernier_06_curios@icloud.com>
Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
* Add support for virtualenv in installer

Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>

* Run workflows when install script is modified

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>

---------

Signed-off-by: Jose Enriquez <alenrqz21@gmail.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Co-authored-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: predat <sylvain@predat.fr>
…oftwareFoundation#1659)

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
…emySoftwareFoundation#1660)

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Bryce Gattis <brycegattis@yahoo.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
…1673)

* Delete rez/utils/json.py

---------

Signed-off-by: Bryce Gattis <brycegattis@yahoo.com>
Signed-off-by: Bryce Gattis <brycegattis@yahoo.com>
…oftwareFoundation#1678)

* remove rez.utils.py23 module
* remove some more __nonzero__ methods

---------

Signed-off-by: wwn <1506514799@qq.com>
Signed-off-by: brycegbrazen <bryce.gattis@brazenanimation.com>
Signed-off-by: brycegbrazen <bryce.gattis@brazenanimation.com>
Signed-off-by: Stephen Mackenzie <maxnbk@users.noreply.github.com>
…areFoundation#1691)

* Tick off several completed boxes in onboarding document

Signed-off-by: Stephen Mackenzie <maxnbk@users.noreply.github.com>

* Update ASWF/ONBOARDING.md

Signed-off-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>

---------

Signed-off-by: Stephen Mackenzie <maxnbk@users.noreply.github.com>
Signed-off-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
…Foundation#1666)

This will fix issues when rez-cp could fail in some situations on WIndows.

Signed-off-by: loonghao <hal.long@outlook.com>
AcademySoftwareFoundation#1672)

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Previously, package orderers were only applied on packages. This was an oversight. With this change, they are now applied to variants too.

---------

Signed-off-by: Ben Andersen <ben@isohedron.com.au>
Co-authored-by: Paul Molodowitch <elrond79@gmail.com>
Signed-off-by: Bryce Gattis <brycegattis@yahoo.com>
…wareFoundation#1677)

Improve clarify of early binding functions documentation by documenting what is accessible within an early function.

Signed-off-by: Bryce Gattis <brycegattis@yahoo.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Jer <samson-jerome@users.noreply.github.com>
…on#1692)

Signed-off-by: brycegbrazen <bryce.gattis@brazenanimation.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Co-authored-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Pixel-Minions and others added 20 commits September 26, 2024 17:27
…twareFoundation#1781)

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
…1785)

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.1 to 4.1.7.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.1.1...692973e)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#1784)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.2 to 4.3.3.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4.3.2...6546280)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#1783)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.25.10 to 3.25.11.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@23acc5c...b611370)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#1790)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.3 to 4.3.4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@6546280...0b2256b)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…undation#1789)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4.1.7 to 4.1.8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@65a9edc...fa0a91b)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#1795)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.25.11 to 3.25.12.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@b611370...4fa2a79)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ion#1794)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5.1.0 to 5.1.1.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@82c7e63...39cd149)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ySoftwareFoundation#1798)

Signed-off-by: Ruzette Krukkert <ruzettekrukkert@fusefx.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Co-authored-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
…dation#1816)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.25.12 to 3.26.5.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@4fa2a79...2c779ab)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#1814)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.4 to 4.3.6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@0b2256b...834a144)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tion#1805)

Bumps [ossf/scorecard-action](https://github.com/ossf/scorecard-action) from 2.3.3 to 2.4.0.
- [Release notes](https://github.com/ossf/scorecard-action/releases)
- [Changelog](https://github.com/ossf/scorecard-action/blob/main/RELEASE.md)
- [Commits](ossf/scorecard-action@dc50aa9...62b2cac)

---
updated-dependencies:
- dependency-name: ossf/scorecard-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ation#1827)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.5 to 3.26.7.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@2c779ab...8214744)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ion#1820)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5.1.1 to 5.2.0.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@39cd149...f677139)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#1819)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.6 to 4.4.0.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@834a144...5076954)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ocs (AcademySoftwareFoundation#1804)

Updates the requirements on [sphinx](https://github.com/sphinx-doc/sphinx) to permit the latest version.
- [Release notes](https://github.com/sphinx-doc/sphinx/releases)
- [Changelog](https://github.com/sphinx-doc/sphinx/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinx@v7.0.0...v8.0.0)

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

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…areFoundation#1778)

Fixes issue where powershell plugins were not correctly capturing exit codes when aliased commands failed (AcademySoftwareFoundation#1778)

---------

Signed-off-by: brycegbrazen <bryce.gattis@brazenanimation.com>
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
…rip. Also bumped the serialization formation from 4.7 to 4.8 (AcademySoftwareFoundation#1810)

Signed-off-by: Nathan Rusch <nathan.rusch@scanlinevfx.com>
- Introduced `--pkg-cache-mode` argument to specify caching mode (`sync` or `async`).
- Updated `add_variant` function to handle the new caching modes.
@Pixel-Minions
Copy link
Contributor Author

@JeanChristopheMorinPerso For review

@JeanChristopheMorinPerso
Copy link
Member

@Pixel-Minions I'm not too sure how you created your branch, but there is a lot of commits that are not from you and DCO fails because of that.

@JeanChristopheMorinPerso JeanChristopheMorinPerso added the Blocked by DCO Waiting on commit signoff label Oct 13, 2024
@JeanChristopheMorinPerso JeanChristopheMorinPerso added this to the Next milestone Oct 13, 2024
@JeanChristopheMorinPerso JeanChristopheMorinPerso changed the title Add support for synchronous and asynchronous package caching modes Add support for synchronous mode in rez-pkg-cache Oct 13, 2024
@JeanChristopheMorinPerso
Copy link
Member

Closing in favor of #1853

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked by DCO Waiting on commit signoff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants