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

[tests,pa,spm] add/fix configurations to enable running integration tests #26

Merged

Conversation

timothytrippel
Copy link
Collaborator

@timothytrippel timothytrippel commented Oct 1, 2024

The ./run_integration_tests.sh script was failing for many reasons, and this fixes two issues:

  1. adding a reference SKU authorization YAML config file with a test authorization password to revive the pa/loadtest.go program, and
  2. default to disabling the registry buffer until [pb] enable connection to proxybuffer service #24 can be addressed.

This partially addresses #17.

The infrastructure had been updated to parse a SKU config file that
contains a hashed authentication secret (password), however a reference
SKU auth file was missing. This adds one to running the integration
load test.

Signed-off-by: Tim Trippel <[email protected]>
The registry buffer service was removed, and will soon be replaced with
the proxybuffer service. However, the enablement flag within the PA was
still defaulting to true. This is updated to default to false until lowRISC#24
can be fixed.

Signed-off-by: Tim Trippel <[email protected]>
@timothytrippel timothytrippel merged commit 90c6a96 into lowRISC:main Oct 1, 2024
3 checks passed
@timothytrippel timothytrippel deleted the fix-integration-tests-pt1 branch October 1, 2024 21:45
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