-
Notifications
You must be signed in to change notification settings - Fork 217
Customizing Docker Script Executors
Nolan Ehrstrom edited this page Apr 30, 2020
·
25 revisions
Issue: https://github.com/ProcessMaker/processmaker/issues/2887
Video: https://drive.google.com/file/d/1RTh4I8SzF8XjrFXBZJB7ZU1XOQmUGKrW/view
-
The following PRs must be merged
- Core: https://github.com/ProcessMaker/processmaker/pull/2971
-
php
https://github.com/ProcessMaker/docker-executor-php/pull/6 -
lua
https://github.com/ProcessMaker/docker-executor-lua/issues/3 -
node
https://github.com/ProcessMaker/docker-executor-node/pull/3 -
python
- No PR, updated on develop -
java
https://github.com/ProcessMaker/docker-executor-java/pull/2 -
csharp
https://github.com/ProcessMaker/docker-executor-csharp/pull/3
-
Put php, lua, and node on Packagist. Republish the others.
-
Update composer.json versions for php, lua, node after published to packagist.
-
Remove "Temporary until packages are deployed" commands in .circleci/config.yml
- Make sure there are no files in core (except /storage) that should not be there. Some PHP files were removed in 4.0.4
- Make sure the data sources and saved search packages are the newest versions.
- Clear cache
php artisan optimize:clear
- Core now has 3 new packages specified in composer.json: 'docker-executor-php', 'docker-executor-lua', 'docker-executor-node'
- Run
composer install
to get these new packages installed. - Run the installers for each installed language. At a minimum, that will be php, lua, and node. Check if the instance has other
docker-executor-*
packages installed. The installer will need to get run for each installed language.php artisan docker-executor-php:install php artisan docker-executor-lua:install php artisan docker-executor-node:install php artisan docker-executor-java:install php artisan docker-executor-csharp:install php artisan docker-executor-python:install
- Verify that base images were automatically built and deployed to docker hub as processmaker4/executor-{lang}
- Install any executors you would like on the instance with
composer require processmaker/docker-executor-{lang}
- The install script for each language must be re-run from core, even if the language was previously installed. For example:
php artisan docker-executor-php:install
In an instance of PM4, click on Admin on the top menu, then Script Executors on the side menu.
Add any docker commands to customize the executor. For example, in the PHP script executor, you can add the google api by adding the line RUN composer require google/apiclient:^2.0
- C# SDK will not build. FilesApi.cs and RequestFilesApi.cs are attempting to send a
List<System.IO.Stream>
toParameterToFile()
which only acceptsSystem.IO.Stream
. I don't know if this is an annotation problem or a bug in openapi. SDK has been commented out in the default dockerfile for the csharp package. Issue to enable it later: https://github.com/ProcessMaker/docker-executor-csharp/issues/1 - SDK Builder is using openapitools/openapi-generator-cli:v4.2.2 (set in ProcessMaker/BuildSdk.php). v4.2.3 is available but breaks the Java sdk. (v4.2.3 does not seem to fix the above C# issue anyway)