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

Support tag the access source #135

Merged
merged 2 commits into from
Sep 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,9 @@ readme = "README.md"

[tool.poetry.dependencies]
python = "^3.9"
fsspec = "==2023.5.0"
fsspec = ">=2023.5.0"
tos = ">=2.7.0"
volcengine= "==1.0.154"

[tool.poetry.group.dev.dependencies]
fsspec = ">=2023.5.0"
Expand All @@ -22,6 +23,7 @@ pytest-cov = "==5.0.0"
coverage = "==7.5.0"
ruff = "==0.6.0"
types-requests = "==2.32.0.20240907"
volcengine= "==1.0.154"

[tool.pydocstyle]
convention = "numpy"
Expand Down Expand Up @@ -64,6 +66,7 @@ select = [
ignore = [
"S101", # Use of `assert` detected
"S311", # Standard pseudo-random generators are not suitable for cryptographic purposes
"S108", # Probable insecure usage of temporary file or directory
"D203", # no-blank-line-before-class
"D213", # multi-line-summary-second-line
"PLR0913", # Too many arguments in function definition
Expand Down
4 changes: 3 additions & 1 deletion tosfs/consts.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@

LS_OPERATION_DEFAULT_MAX_ITEMS = 1000

TOSFS_LOG_FORMAT = "%(asctime)s %(name)s [%(levelname)s] %(filename)s:%(lineno)d %(funcName)s : %(message)s" # noqa: E501

# environment variable names
ENV_NAME_TOSFS_LOGGING_LEVEL = "TOSFS_LOGGING_LEVEL"
TOSFS_LOG_FORMAT = "%(asctime)s %(name)s [%(levelname)s] %(filename)s:%(lineno)d %(funcName)s : %(message)s" # noqa: E501
ENV_NAME_TOS_BUCKET_TAG_ENABLE = "TOS_BUCKET_TAG_ENABLE"
20 changes: 20 additions & 0 deletions tosfs/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
from fsspec import AbstractFileSystem
from fsspec.spec import AbstractBufferedFile
from fsspec.utils import setup_logging as setup_logger
from tos.auth import CredentialProviderAuth
from tos.exceptions import TosClientError, TosServerError
from tos.models import CommonPrefixInfo
from tos.models2 import (
Expand Down Expand Up @@ -54,6 +55,7 @@
from tosfs.fsspec_utils import glob_translate
from tosfs.mpu import MultipartUploader
from tosfs.retry import retryable_func_executor
from tosfs.tag import BucketTagMgr
from tosfs.utils import find_bucket_key, get_brange

logger = logging.getLogger("tosfs")
Expand Down Expand Up @@ -203,6 +205,10 @@ def __init__(
if version_aware:
raise ValueError("Currently, version_aware is not supported.")

self.tag_enabled = os.environ.get("TOS_TAG_ENABLED", True)
if self.tag_enabled:
self._init_tag_manager()

self.version_aware = version_aware
self.default_block_size = (
default_block_size or FILE_OPERATION_READ_WRITE_BUFFER_SIZE
Expand Down Expand Up @@ -2093,12 +2099,26 @@ def _split_path(self, path: str) -> Tuple[str, str, Optional[str]]:

bucket, keypart = find_bucket_key(path)
key, _, version_id = keypart.partition("?versionId=")

if self.tag_enabled:
self.bucket_tag_mgr.add_bucket_tag(bucket)

return (
bucket,
key,
version_id if self.version_aware and version_id else None,
)

def _init_tag_manager(self) -> None:
auth = self.tos_client.auth
if isinstance(auth, CredentialProviderAuth):
credentials = auth.credentials_provider.get_credentials()
self.bucket_tag_mgr = BucketTagMgr(
credentials.get_ak(), credentials.get_sk(), auth.region
)
else:
raise TosfsError("Currently only support CredentialProviderAuth type")

@staticmethod
def _fill_dir_info(
bucket: str, common_prefix: Optional[CommonPrefixInfo], key: str = ""
Expand Down
228 changes: 228 additions & 0 deletions tosfs/tag.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,228 @@
# ByteDance Volcengine EMR, Copyright 2024.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

"""The module contains all the business logic for tagging tos buckets ."""

import fcntl
import functools
import json
import logging
import os
import threading
from concurrent.futures import ThreadPoolExecutor
from typing import Any

from volcengine.ApiInfo import ApiInfo
from volcengine.base.Service import Service
from volcengine.Credentials import Credentials
from volcengine.ServiceInfo import ServiceInfo

PUT_TAG_ACTION_NAME = "PutBucketDoubleMeterTagging"
GET_TAG_ACTION_NAME = "GetBucketTagging"
EMR_OPEN_API_VERSION = "2022-12-29"
OPEN_API_HOST = "open.volcengineapi.com"
ACCEPT_HEADER_KEY = "accept"
ACCEPT_HEADER_JSON_VALUE = "application/json"

THREAD_POOL_SIZE = 2
TAGGED_BUCKETS_FILE = "/tmp/.emr_tagged_buckets"

CONNECTION_TIMEOUT_DEFAULT_SECONDS = 60 * 5
SOCKET_TIMEOUT_DEFAULT_SECONDS = 60 * 5

service_info_map = {
"cn-beijing": ServiceInfo(
OPEN_API_HOST,
{
ACCEPT_HEADER_KEY: ACCEPT_HEADER_JSON_VALUE,
},
Credentials("", "", "emr", "cn-beijing"),
CONNECTION_TIMEOUT_DEFAULT_SECONDS,
SOCKET_TIMEOUT_DEFAULT_SECONDS,
"http",
),
"cn-guangzhou": ServiceInfo(
OPEN_API_HOST,
{
ACCEPT_HEADER_KEY: ACCEPT_HEADER_JSON_VALUE,
},
Credentials("", "", "emr", "cn-guangzhou"),
CONNECTION_TIMEOUT_DEFAULT_SECONDS,
SOCKET_TIMEOUT_DEFAULT_SECONDS,
"http",
),
"cn-shanghai": ServiceInfo(
OPEN_API_HOST,
{
ACCEPT_HEADER_KEY: ACCEPT_HEADER_JSON_VALUE,
},
Credentials("", "", "emr", "cn-shanghai"),
CONNECTION_TIMEOUT_DEFAULT_SECONDS,
SOCKET_TIMEOUT_DEFAULT_SECONDS,
"http",
),
"ap-southeast-1": ServiceInfo(
OPEN_API_HOST,
{
ACCEPT_HEADER_KEY: ACCEPT_HEADER_JSON_VALUE,
},
Credentials("", "", "emr", "ap-southeast-1"),
CONNECTION_TIMEOUT_DEFAULT_SECONDS,
SOCKET_TIMEOUT_DEFAULT_SECONDS,
"http",
),
"cn-beijing-qa": ServiceInfo(
OPEN_API_HOST,
{
ACCEPT_HEADER_KEY: ACCEPT_HEADER_JSON_VALUE,
},
Credentials("", "", "emr_qa", "cn-beijing"),
CONNECTION_TIMEOUT_DEFAULT_SECONDS,
SOCKET_TIMEOUT_DEFAULT_SECONDS,
"http",
),
}

api_info = {
PUT_TAG_ACTION_NAME: ApiInfo(
"POST",
"/",
{"Action": PUT_TAG_ACTION_NAME, "Version": EMR_OPEN_API_VERSION},
{},
{},
),
GET_TAG_ACTION_NAME: ApiInfo(
"GET",
"/",
{"Action": GET_TAG_ACTION_NAME, "Version": EMR_OPEN_API_VERSION},
{},
{},
),
}


class BucketTagAction(Service):
"""BucketTagAction is a class to manage the tag of bucket."""

_instance_lock = threading.Lock()

def __new__(cls, *args: Any, **kwargs: Any) -> Any:
"""Singleton."""
if not hasattr(BucketTagAction, "_instance"):
with BucketTagAction._instance_lock:
if not hasattr(BucketTagAction, "_instance"):
BucketTagAction._instance = object.__new__(cls)
return BucketTagAction._instance

def __init__(self, key: str, secret: str, region: str) -> None:
"""Init BucketTagAction."""
super().__init__(self.get_service_info(region), self.get_api_info())
self.set_ak(key)
self.set_sk(secret)

@staticmethod
def get_api_info() -> dict:
"""Get api info."""
return api_info

@staticmethod
def get_service_info(region: str) -> ServiceInfo:
"""Get service info."""
service_info = service_info_map.get(region)
if service_info:
return service_info

raise Exception(f"Do not support region: {region}")

def put_bucket_tag(self, bucket: str) -> tuple[str, bool]:
"""Put tag for bucket."""
params = {
"Bucket": bucket,
}

try:
res = self.json(PUT_TAG_ACTION_NAME, params, json.dumps(""))
res_json = json.loads(res)
logging.debug("Put tag for bucket %s successfully: %s .", bucket, res_json)
return bucket, True
except Exception as e:
logging.debug("Put tag for bucket %s failed: %s .", bucket, e)
return bucket, False

def get_bucket_tag(self, bucket: str) -> bool:
"""Get tag for bucket."""
params = {
"Bucket": bucket,
}
try:
res = self.get(GET_TAG_ACTION_NAME, params)
res_json = json.loads(res)
logging.debug("The get bucket tag's response is %s", res_json)
return True
except Exception as e:
logging.debug("Get tag for %s is failed: %s", bucket, e)
return False


def singleton(cls: Any) -> Any:
"""Singleton decorator."""
_instances = {}

@functools.wraps(cls)
def get_instance(*args: Any, **kwargs: Any) -> Any:
if cls not in _instances:
_instances[cls] = cls(*args, **kwargs)
return _instances[cls]

return get_instance


@singleton
class BucketTagMgr:
"""BucketTagMgr is a class to manage the tag of bucket."""

def __init__(self, key: str, secret: str, region: str):
"""Init BucketTagMgr."""
self.executor = ThreadPoolExecutor(max_workers=THREAD_POOL_SIZE)
self.cached_bucket_set: set = set()
self.key = key
self.secret = secret
self.region = region
self.bucket_tag_service = BucketTagAction(self.key, self.secret, self.region)

def add_bucket_tag(self, bucket: str) -> None:
"""Add tag for bucket."""
collect_bucket_set = {bucket}

if not collect_bucket_set - self.cached_bucket_set:
return

if os.path.exists(TAGGED_BUCKETS_FILE):
with open(TAGGED_BUCKETS_FILE, "r") as file:
tagged_bucket_from_file_set = set(file.read().split(" "))
self.cached_bucket_set |= tagged_bucket_from_file_set

need_tag_buckets = collect_bucket_set - self.cached_bucket_set

for res in self.executor.map(
self.bucket_tag_service.put_bucket_tag, need_tag_buckets
):
if res[1]:
self.cached_bucket_set.add(res[0])

with open(TAGGED_BUCKETS_FILE, "w") as fw:
fcntl.flock(fw, fcntl.LOCK_EX)
fw.write(" ".join(self.cached_bucket_set))
fcntl.flock(fw, fcntl.LOCK_UN)
fw.close()
46 changes: 46 additions & 0 deletions tosfs/tests/test_tag.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
# ByteDance Volcengine EMR, Copyright 2024.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

import os
from time import sleep

import pytest

from tosfs.tag import TAGGED_BUCKETS_FILE


@pytest.fixture
def _prepare_tag_env():
if os.path.exists(TAGGED_BUCKETS_FILE):
os.remove(TAGGED_BUCKETS_FILE)
yield
if os.path.exists(TAGGED_BUCKETS_FILE):
os.remove(TAGGED_BUCKETS_FILE)


@pytest.mark.usefixtures("_prepare_tag_env")
def test_bucket_tag_action(tosfs, bucket, temporary_workspace):
tag_mgr = tosfs.bucket_tag_mgr
if tag_mgr is None:
return

tag_mgr.cached_bucket_set = set()
tag_mgr.add_bucket_tag(bucket)
sleep(10)
assert os.path.exists(TAGGED_BUCKETS_FILE)
with open(TAGGED_BUCKETS_FILE, "r") as f:
tagged_buckets = f.read()
assert bucket in tagged_buckets

assert tag_mgr.bucket_tag_service.get_bucket_tag(bucket)