Skip to content

Commit

Permalink
Merge pull request #56 from lsst-dm/tickets/DM-44847a
Browse files Browse the repository at this point in the history
DM-44847a: (hotfix) Move profile to enqueue.
  • Loading branch information
ktlim authored Jun 19, 2024
2 parents 518233b + fcde6a3 commit 23b793d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 9 deletions.
2 changes: 1 addition & 1 deletion Dockerfile.ingest
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
ARG RUBINENV_VERSION=8.0.0
FROM lsstsqre/newinstall:${RUBINENV_VERSION}
ARG OBS_LSST_VERSION
ENV OBS_LSST_VERSION=${OBS_LSST_VERSION:-w_2024_12}
ENV OBS_LSST_VERSION=${OBS_LSST_VERSION:-w_2024_24}
USER lsst
RUN source loadLSST.bash && mamba install redis-py rucio-clients
RUN source loadLSST.bash && eups distrib install -t "${OBS_LSST_VERSION}" lsst_obs
Expand Down
5 changes: 4 additions & 1 deletion src/enqueue.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,9 @@
r = setup_redis()
notification_secret = os.environ["NOTIFICATION_SECRET"]
regexp = re.compile(os.environ.get("DATASET_REGEXP", r"fits$"))
profile = os.environ.get("PROFILE", "")
if profile != "":
profile += "@"


def enqueue_objects(objects):
Expand Down Expand Up @@ -119,7 +122,7 @@ def notify():
logger.info("Unrecognized secret %s", r["opaqueData"])
continue
object_names.append(
r["s3"]["bucket"]["name"] + "/" + urllib.parse.unquote_plus(r["s3"]["object"]["key"])
profile + r["s3"]["bucket"]["name"] + "/" + urllib.parse.unquote_plus(r["s3"]["object"]["key"])
)
info_list = enqueue_objects(object_names)
update_stats(info_list)
Expand Down
8 changes: 1 addition & 7 deletions src/ingest.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@
logger = setup_logging(__name__)
r = setup_redis()
bucket = os.environ["BUCKET"]
if (pos := bucket.find("@")) >= 0:
pos += 1
profile = bucket[:pos]
bucket = bucket[pos:]
else:
profile = ""
if bucket.startswith("rubin:"):
os.environ["LSST_DISABLE_BUCKET_VALIDATION"] = "1"
redis_queue = f"QUEUE:{bucket}"
Expand Down Expand Up @@ -205,7 +199,7 @@ def main():
# Process any entries on the worker queue.
if r.llen(worker_queue) > 0:
blobs = r.lrange(worker_queue, 0, -1)
resources = [ResourcePath(f"s3://{profile}{b.decode()}") for b in blobs]
resources = [ResourcePath(f"s3://{b.decode()}") for b in blobs]

# Ingest if we have resources
if resources:
Expand Down

0 comments on commit 23b793d

Please sign in to comment.