aws-deadline / deadline-cloud-for-maya

AWS Deadline Cloud for Maya
Apache License 2.0
11 stars 12 forks source link

feat!: Create a script to build adaptor packaging artifacts #87

Closed mwiebe closed 7 months ago

mwiebe commented 7 months ago

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

We have designed a new structure for how we lay out application interface adaptors into packages. In order to simplify the packaging step, we'll put a script that generates an artifact with that layout into each adaptor git repo.

In order to perform the layout we wanted, I needed to modify the adaptor runtime. Those changes are in https://github.com/OpenJobDescription/openjd-adaptor-runtime-for-python/pull/69

The adaptor runtime dependency was out of date, After updating, it required changes from a socket_path to a server_path parameter.

What was the solution? (How)

What is the impact of this change?

The repo has a script that can generate a .tar archive containing a prefix layout of the adaptor python library and its dependencies, along with an entry point.

How was this change tested?

The generated .tar archive was used to create a package, and that was tested on a Deadline Cloud farm.

Did you run the "Job Bundle Output Tests"? If not, why not? If so, paste the test results here.

Yes.


Timestamp: 2024-02-07T23:28:17.849524-08:00
Running job bundle output test: D:\deadline-clients\deadline-cloud-for-maya\job_bundle_output_tests\cube

cube
Test succeeded

Timestamp: 2024-02-07T23:28:18.498383-08:00
Running job bundle output test: D:\deadline-clients\deadline-cloud-for-maya\job_bundle_output_tests\layers

layers
Test succeeded

Timestamp: 2024-02-07T23:28:19.650601-08:00
Running job bundle output test: D:\deadline-clients\deadline-cloud-for-maya\job_bundle_output_tests\layers_no_variation

layers_no_variation
Test succeeded

All tests passed, ran 3 total.
Timestamp: 2024-02-07T23:28:23.355375-08:00

Was this change documented?

No

Is this a breaking change?

Yes, it updates the openjd adaptor runtime dependency which had a breaking change.