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

Run multitenancy tests in LM setups #1350

Draft
wants to merge 4 commits into
base: vpc20-feature-branch
Choose a base branch
from

Conversation

ksamoray
Copy link
Collaborator

@ksamoray ksamoray commented Sep 3, 2024

DO not skip multitenancy tests in LM environments, given that project id is set

@ksamoray
Copy link
Collaborator Author

ksamoray commented Sep 3, 2024

/test-all

@ksamoray ksamoray marked this pull request as draft September 3, 2024 16:03
This test creates context profile and shares it with the VPC
Signed-off-by: Anna Khmelnitsky <[email protected]>
Due to a incorrect code, this attributed was defined as read-only
To accomodate this fix, we also need to retrieve session context
from parent resource during getOrGenerateID process

Signed-off-by: Anna Khmelnitsky <[email protected]>
Context is determined fromthe parent share_path
For consistency with other resources, we don't expose the
context block (which provides redundant info) when parent
path is present and required.

Signed-off-by: Anna Khmelnitsky <[email protected]>
@ksamoray ksamoray force-pushed the run_vpc_with_lm branch 7 times, most recently from ee8d47f to a776921 Compare September 15, 2024 14:50
DO not skip multitenancy tests in LM environments, given that project id is set

Signed-off-by: Kobi Samoray <[email protected]>
@ksamoray
Copy link
Collaborator Author

/test-all

2 similar comments
@ksamoray
Copy link
Collaborator Author

/test-all

@ksamoray
Copy link
Collaborator Author

/test-all

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