Skip to content

Commit 5e5ef37

Browse files
authored
Remove Github username from index_name for integration tests (#514)
## Problem A couple of integration tests were failing because the index_name included the GitHub username and a UUID, which caused it to exceed the 45-character limit. ## Solution Remove Github username from the index_name ## Type of Change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update - [x] Infrastructure change (CI configs, etc) - [ ] Non-code change (docs, etc) - [ ] None of the above: (explain here) ## Test Plan Failing integration tests should run successfully now
1 parent 2fe971e commit 5e5ef37

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

tests/integration/helpers/helpers.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -280,11 +280,7 @@ async def asyncio_wait_until(
280280

281281

282282
def default_create_index_params(request, run_id):
283-
github_actor = os.getenv("GITHUB_ACTOR", None)
284-
user = os.getenv("USER", None)
285-
index_owner = github_actor or user or "unknown"
286-
287-
index_name = f"{index_owner}-{str(uuid.uuid4())}"
283+
index_name = f"{str(uuid.uuid4())}"
288284
tags = index_tags(request, run_id)
289285
cloud = get_environment_var("SERVERLESS_CLOUD", "aws")
290286
region = get_environment_var("SERVERLESS_REGION", "us-west-2")

0 commit comments

Comments
 (0)