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

Avoid "Manually specified and authenticated users mismatch" YT error" #28

Conversation

faucct
Copy link
Collaborator

@faucct faucct commented Sep 25, 2024

No description provided.

@faucct faucct force-pushed the feature/avoid-manually-specified-and-authenticated-users-mismatch-yt-error branch from 4cbc191 to 5bd3866 Compare September 25, 2024 13:40
Copy link

robot-magpie bot commented Sep 27, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link
Collaborator

@alextokarew alextokarew left a comment

Choose a reason for hiding this comment

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

Unit and integration tests are failing for this pull request, please fix it

Copy link

robot-magpie bot commented Oct 3, 2024

❌ This pull request has been discarded.

Copy link

robot-magpie bot commented Oct 10, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link
Collaborator

@alextokarew alextokarew left a comment

Choose a reason for hiding this comment

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

Compilation is still failing

Copy link

robot-magpie bot commented Oct 10, 2024

❌ This pull request has been discarded.

Copy link

robot-magpie bot commented Oct 14, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link

robot-magpie bot commented Oct 14, 2024

❌ This pull request has been discarded.

Copy link

robot-magpie bot commented Oct 15, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link

robot-magpie bot commented Oct 15, 2024

❌ This pull request has been discarded.

Copy link

robot-magpie bot commented Oct 15, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link

robot-magpie bot commented Oct 15, 2024

❌ This pull request has been discarded.

@faucct faucct force-pushed the feature/avoid-manually-specified-and-authenticated-users-mismatch-yt-error branch from 7ea4686 to 5eaa5a0 Compare October 15, 2024 13:00
Copy link

robot-magpie bot commented Oct 15, 2024

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link
Collaborator

@alextokarew alextokarew left a comment

Choose a reason for hiding this comment

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

Can you write a test that does the following:

  1. Creates a table with some data
  2. Creates two users on a cluster
  3. Sets an ACL to the created table such that the one user has the permission to select the data from it and the other hasn't
  4. Sequentially read the table using these users, and the first should get the result, the second will raise an exception.

Sample code is here:

create_user("spyt_user")
create_user("rightless_user")
path = "//tmp/secret_table"
create("table", path, attributes={"schema": [{"name": "s", "type": "string"}]})
rows = [{"s": "row"}]
write_table(path, rows, verbose=False)
set(path + "/@acl", [make_ace("allow", "spyt_user", "read"), make_ace("deny", "rightless_user", "read")])
make_query(f"select * from yt.`ytTable:/{path}`", user="rightless_user") # throws an exception
result = make_query(f"select * from yt.`ytTable:/{path}`", user="spyt_user")
assert_items_equal(result, rows)

Copy link

robot-magpie bot commented Oct 21, 2024

❌ This pull request has been discarded.

@faucct faucct closed this Oct 24, 2024
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