Workers do not build anymore

Example of crash : https://preprod.arkindex.teklia.com/process/03114820-9a1e-4642-a743-ff84ed08ac22/0

According to @mlbonhomme on slack it's due to https://gitlab.com/teklia/arkindex/backend/-/commit/66fe35054643d6308ff5bf851eccb8b65d0d98c9

This should probably have been caught by a unit test

changed milestone to %Arkindex 1.3.5

added Bug + 1 deleted label

assigned to @erwanrouchet

This should be the fix https://gitlab.com/teklia/arkindex/tasks/-/merge_requests/330

create_worker_version in the git worker import had this API call with revision:

            version = default_client.request(
                "CreateWorkerVersion",
                id=worker["id"],
                body={
                    "revision": str(self.revision_id),
                    "gpu_usage": worker_config.pop("gpu_usage"),
                    "model_usage": worker_config.pop("model_usage", False),
                    "configuration": worker_config,
                },
            )

I think an underlying issue is that the tests in task entirely mock the API client without using / validating against its actual schema, unlike what's done for the CLI tests for example.

The integration tests in tasks' integration directory are meant to use the actual API. They currently use the prod instance; they could be reconfigured to use the schema from master as the CLI does. This might still not have let us notice this issue as this would require the tasks tests to be run when we make changes in the backend, which requires a multi-project pipeline.

Moved to tasks#168 (closed)

closed