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

fix(sct_config): define SCT_AWS_ACCOUNT_ID in SCTConfiguration #9553

Merged
merged 1 commit into from
Dec 15, 2024

Conversation

mikliapko
Copy link
Contributor

Environmental variable SCT_AWS_ACCOUNT_ID is used in aws_okta.py:

account_id = os.environ.get('SCT_AWS_ACCOUNT_ID', '797456418907')

The fact it's not defined in sct_config.py fails SCT test execution for non-default AWS account, ID of which is passed as env variable:

ValueError: Unsupported environment variables were used: - SCT_AWS_ACCOUNT_ID=400694452550

The described above case is relevant for SCT tests with the cluster created in Scylla Cloud.

Testing

  • None

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Environmental variable SCT_AWS_ACCOUNT_ID is used in aws_okta.py:

`account_id = os.environ.get('SCT_AWS_ACCOUNT_ID', '797456418907')`

The fact it's not defined in sct_config.py fails SCT test execution
for non-default AWS account, ID of which is passed as env variable:

`ValueError: Unsupported environment variables were used: -
 SCT_AWS_ACCOUNT_ID=400694452550`

The described above case is relevant for SCT tests with the cluster
created in Scylla Cloud.
@mikliapko mikliapko self-assigned this Dec 15, 2024
@mikliapko mikliapko requested a review from a team December 15, 2024 09:28
@mikliapko mikliapko added the backport/none Backport is not required label Dec 15, 2024
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

@fruch fruch merged commit 24ac0dd into scylladb:master Dec 15, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/none Backport is not required promoted-to-master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants