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

TEST: CI: CI broken because of 403 HTTPError for taxi CSV on modin-test s3 #5252

Closed
mvashishtha opened this issue Nov 23, 2022 · 5 comments · Fixed by #5257
Closed

TEST: CI: CI broken because of 403 HTTPError for taxi CSV on modin-test s3 #5252

mvashishtha opened this issue Nov 23, 2022 · 5 comments · Fixed by #5257
Assignees
Labels
P0 Highest priority tasks requiring immediate fix

Comments

@mvashishtha
Copy link
Collaborator

mvashishtha commented Nov 23, 2022

I can't reproduce this failure locally. It appears in CI for at least two unrelated PRs: https://github.com/modin-project/modin/actions/runs/3527986556/jobs/5918024859

cc @noloerino

@mvashishtha mvashishtha added the P0 Highest priority tasks requiring immediate fix label Nov 23, 2022
@RehanSD
Copy link
Collaborator

RehanSD commented Nov 23, 2022

Hmm - I tried using curl as well as going to the link thats failing directly, and I'm getting an error that all access is forbidden. Seems that this bucket is one owned by us, so let me check around to see whats going on with it!

@mvashishtha
Copy link
Collaborator Author

mvashishtha commented Nov 23, 2022

Some questions:

  • Who owns the modin-test bucket?
  • Why does the test fail in CI but not locally? Do we use a local taxi.csv locally?
  • Why can I do aws s3 ls modin-datasets but not aws s3 ls modin-test? The former gives me An error occurred (AllAccessDisabled) when calling the ListObjectsV2 operation: All access to this object has been disabled. Is that the root cause?

cc @modin-project/modin-contributors @modin-project/modin-core @gshimansky

@RehanSD
Copy link
Collaborator

RehanSD commented Nov 23, 2022

@mvashishtha I'm able to repro the failure locally - I think maybe you have a cached copy of the dataset? I'm checking with @devin-petersohn to see who has ownership of the S3 buckets, since I think he may be the owner!

@mvashishtha
Copy link
Collaborator Author

@RehanSD yep, I can reproduce the error when I remove the taxi.csv at my modin root.

@RehanSD
Copy link
Collaborator

RehanSD commented Nov 23, 2022

Seems like the bucket's access policy has changed, will reach out to see if they can be changed back or if we need to self host a bucket.

RehanSD added a commit to RehanSD/modin that referenced this issue Nov 23, 2022
…ssues are resolved for `modin-test` bucket

Signed-off-by: Rehan Durrani <rehan@ponder.io>
mvashishtha pushed a commit that referenced this issue Nov 23, 2022
…olved for `modin-test` bucket (#5257)

Signed-off-by: Rehan Durrani <rehan@ponder.io>
dchigarev pushed a commit that referenced this issue Nov 25, 2022
…olved for `modin-test` bucket (#5257)

Signed-off-by: Rehan Durrani <rehan@ponder.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Highest priority tasks requiring immediate fix
Projects
None yet
2 participants