Skip to content

feat: migrate to job attachments package#1133

Open
epmog wants to merge 1 commit intoaws-deadline:mainlinefrom
epmog:use_deadline_job_attachments
Open

feat: migrate to job attachments package#1133
epmog wants to merge 1 commit intoaws-deadline:mainlinefrom
epmog:use_deadline_job_attachments

Conversation

@epmog
Copy link
Copy Markdown
Contributor

@epmog epmog commented Apr 23, 2026

What was the problem/requirement? (What/Why)

job-attachments has moved to its own repo and pypi package:

  1. https://github.com/aws-deadline/deadline-cloud-job-attachments
  2. https://pypi.org/project/deadline-job-attachments/

and deadline-cloud needs to be updated to consume the new library.

What was the solution? (How)

Perform the second part of the migration, which is delete all the previous job attachments specific code from this repo

What is the impact of this change?

Should be invisible to users, but now we have a dedicated library with its own version/release cycle.

How was this change tested?

hatch run fmt
hatch run test

Was this change documented?

The github website now pulls job attachments docs from the new job attachments repo.

So now we can delete those docs from this repo

Does this PR introduce new dependencies?

It does, but it's ours :)

Is this a breaking change?

No, because the job attachments library maintained the same interface. It's the same code, just in a new spot.

Does this change impact security?

N/A


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions github-actions Bot added the waiting-on-maintainers Waiting on the maintainers to review. label Apr 23, 2026
@epmog epmog force-pushed the use_deadline_job_attachments branch 4 times, most recently from 09d0c88 to 48aaf98 Compare April 24, 2026 00:17
Signed-off-by: Morgan Epp <60796713+epmog@users.noreply.github.com>
@epmog epmog force-pushed the use_deadline_job_attachments branch from 48aaf98 to ee3c415 Compare April 24, 2026 22:31
@epmog epmog marked this pull request as ready for review April 24, 2026 22:35
@epmog epmog requested review from a team as code owners April 24, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-on-maintainers Waiting on the maintainers to review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants