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

chore: update registry1 weekly to 0.13.1, API, and testing workflows #1101

Merged
merged 66 commits into from
Oct 1, 2024

Conversation

justinthelaw
Copy link
Contributor

@justinthelaw justinthelaw commented Sep 23, 2024

Description

Updates everything Registry1 to v0.13.0, and fixes the Registry1 weekly to pin all components to 0.13.1 (or whatever the LEAPFROGAI_VERSION is) so proper testing can be performed. Removed integration testing, since it uses a local Python instance of the API, which is out-of-scope. Adds a zarf-config.yaml for non-upstream flavor builds to centralize un-mutable versioning.

Fixes release-please-config.json to also update DEVELOPMENT.md, where it references the LeapfrogAI latest tag as part of an example code block. Adds Chainguard login so we are not rate-limited, and adds granular permissions blocks for the associated workflows. Centralizes model name in the conftest.py and generation configurations within the test_llm_generation.py. Adds better clean Makefile target for all cached test artifacts, and improves Makefile build pattern for registry1 flavors.

BREAKING CHANGES

  • uses a zarf-config.yaml when building the API registry1 flavor to centralize un-mutable version (only mutates via release-please)
    • makes it easier to do registry1 (uses the v) versus upstream flavor images by variabilizing the version number

CHANGES

  • bumps all registry1 deps to 0.13.1
  • changes name of workflow to match pattern of [frequency]-[purpose].yaml
  • fixes pinning and testing technique of the Registry1 image(s)
  • remove out-of-scope integration testing from the Registry1 weekly
  • fixes release-please config to also update DEVELOPMENT.md code block
  • adds chainguard login to the UDS setup tasks
  • modifies and uses the utils.client.py for all e2e tests
  • centralizes generation params for the tests within test_llm_generation.py
  • adds better clean Makefile target for all cached test artifacts
  • improves Makefile build pattern for registry1 flavor
  • removes the udsCliVersion and instead allows the default UDS CLI version packaged with the UDS Common action, allowing renovate to update that automatically, instead of us doing it manually
  • adds env LEAPFROGAI_MODEL: llama-cpp-python to e2e tests that need a specific model identified

Related Issue

Relates to https://repo1.dso.mil/dsop/opensource/defenseunicorns/leapfrogai/api/-/issues/22

Checklist before merging

Copy link

netlify bot commented Sep 23, 2024

Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit 4bd7b38
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/66fc7ef0743fa20008f3e5b9

@justinthelaw justinthelaw changed the title chore: update registry1 weekly for 0.13.0 and fix versioning chore: update registry1 weekly for 0.13.0 and test workflows Sep 23, 2024
@justinthelaw justinthelaw changed the title chore: update registry1 weekly for 0.13.0 and test workflows chore: update registry1 weekly for 0.13.0, testing and test workflows Sep 23, 2024
@justinthelaw justinthelaw changed the title chore: update registry1 weekly for 0.13.0, testing and test workflows chore: update registry1 weekly to 0.13.0 and testing workflows Sep 23, 2024
@justinthelaw justinthelaw marked this pull request as draft September 24, 2024 16:01
gphorvath
gphorvath previously approved these changes Oct 1, 2024
gphorvath
gphorvath previously approved these changes Oct 1, 2024
tests/utils/client.py Outdated Show resolved Hide resolved
tests/utils/client.py Outdated Show resolved Hide resolved
tests/utils/client.py Outdated Show resolved Hide resolved
@justinthelaw justinthelaw merged commit 022e98d into main Oct 1, 2024
32 checks passed
@justinthelaw justinthelaw deleted the chore-update-registry1-weekly-bundle-0.13.0 branch October 1, 2024 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants