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

Set up a scheduled job that runs ORT on ORT #5696

Open
sschuberth opened this issue Aug 26, 2022 · 2 comments
Open

Set up a scheduled job that runs ORT on ORT #5696

sschuberth opened this issue Aug 26, 2022 · 2 comments
Labels
configuration About configuration topics enhancement Issues that are considered to be enhancements

Comments

@sschuberth
Copy link
Member

For the sake of "eating our own dog food" and to showcase how ORT performs on a rather big / complex project like ORT itself, we should run ORT on ORT regularly. This would also get us any compliance documents / artifacts that are eventually needed for distributing ORT as a Docker image.

Several contributors have already worked on this topic on and off, and there is an .ort.yml file for ORT, but there are more problems to solve , probably mostly in tests assets (some of which are deliberately broken). So we should get this straight once, and then ensure "dogfeeding" ORT to ORT works on a regular basis.

@sschuberth sschuberth added enhancement Issues that are considered to be enhancements configuration About configuration topics labels Aug 26, 2022
@nicorikken
Copy link
Member

Bootstrapping dependency order discussed on the 25th of August:

  1. Containerized ORT version for the ORT scanning Action
  2. ORT scanning GitHub Action
  3. Integration of the ORT scanning Action in the ORT GitHub Actions pipeline
  4. Speed up scans using caching to prevent time-out
  5. Provide curations and fix issues to make it not fail

Going by the experiences discussed ORT currently requires quite some curations and still there remain Gradle errors.

The caching is likely to be necessary as the scan would otherwise take too long and would time-out. So some (public) caching method is needed to speed things up.

@sschuberth
Copy link
Member Author

The caching is likely to be necessary as the scan would otherwise take too long and would time-out. So some (public) caching method is needed to speed things up.

Correct. Ideas that were discussed include using ClearlyDefined as a global scan storage (required fixing of #4540) or implementing a new scan storage based on IPFS (see #1328).

sschuberth added a commit that referenced this issue Sep 15, 2022
See [1]. Do this for (later) comparision with scanning ORT with ORT [2].

[1]: https://www.jetbrains.com/help/qodana/qodana-yaml.html#License+audit+configuration
[2]: #5696

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
sschuberth added a commit that referenced this issue Sep 15, 2022
See [1]. Do this for (later) comparision with scanning ORT with ORT [2].

[1]: https://www.jetbrains.com/help/qodana/qodana-yaml.html#License+audit+configuration
[2]: #5696

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
sschuberth added a commit that referenced this issue Sep 15, 2022
See [1]. Do this for (later) comparision with scanning ORT with ORT [2].

[1]: https://www.jetbrains.com/help/qodana/qodana-yaml.html#License+audit+configuration
[2]: #5696

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
sschuberth added a commit that referenced this issue Sep 16, 2022
See [1]. Do this for (later) comparision with scanning ORT with ORT [2].

[1]: https://www.jetbrains.com/help/qodana/qodana-yaml.html#License+audit+configuration
[2]: #5696

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configuration About configuration topics enhancement Issues that are considered to be enhancements
Projects
None yet
Development

No branches or pull requests

2 participants