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

feat: point the integs at a remote trustification server #424

Merged
merged 3 commits into from
Aug 23, 2023

Conversation

jcrossley3
Copy link
Contributor

@jcrossley3 jcrossley3 commented Aug 22, 2023

Fixes #401

Setting the TRUST_URL env var to a remote trustification server triggers the integs to be run against it.

If it's set, other env vars will be required:

  • TRUST_USER_ID -- the client id of the user
  • TRUST_MANAGER_ID -- the client id of the manager
  • TRUST_SECRET -- the secret is assumed to be the same for user & mgr

If KAFKA_BOOTSTRAP_SERVERS is set, its value will be used to configure the event bus. Otherwise, SQS is assumed and valid AWS credentials will be required.

Signed-off-by: Jim Crossley <jim@crossleys.org>
Better to see either sbom, vex, or spog in the names, I think

Signed-off-by: Jim Crossley <jim@crossleys.org>
Fixes trustification#401

Setting the TRUST_URL env var to a remote trustification server
triggers the integs to be run against it.

If it's set, other env vars will be required:
  TRUST_USER_ID -- the client id of the user
  TRUST_MANAGER_ID -- the client id of the manager
  TRUST_SECRET -- the secret is assumed to be the same for user & mgr

If KAFKA_BOOTSTRAP_SERVERS is set, its value will be used to configure
the event bus. Otherwise, SQS is assumed and valid AWS credentials
will be required.

Signed-off-by: Jim Crossley <jim@crossleys.org>
@trust-git-bot
Copy link

🚀 Deployed Preview: https://trustification-spog-pr-424-preview.surge.sh

impl Config {
pub async fn new() -> Self {
let _ = env_logger::try_init();
match std::env::var("TRUST_URL") {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@jcrossley3 This env variable is configured on .env or local?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The .env file is only used by the compose scripts. TRUST_URL needs to be set before running the integs with cargo, e.g.

TRUST_URL=http://localhost:8084 cargo test -p integration-tests

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks for the information

@ctron ctron merged commit 918c159 into trustification:main Aug 23, 2023
13 checks passed
jcrossley3 added a commit to jcrossley3/trustification that referenced this pull request Aug 23, 2023
ref trustification#401 & trustification#424

Signed-off-by: Jim Crossley <jim@crossleys.org>
@jcrossley3 jcrossley3 deleted the 401 branch August 23, 2023 14:17
jcrossley3 added a commit that referenced this pull request Aug 23, 2023
ref #401 & #424

Signed-off-by: Jim Crossley <jim@crossleys.org>
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.

Running integration tests against an existing environment
4 participants