~sircmpwn/builds.sr.ht#204: Manifests started without execute set

Currently manifests that have execute: false set are executed anyway.

Request:

POST http://builds.sr.ht/api/jobs HTTP/1.1

{"manifest":"image: alpine\/edge\npackages:\n    - python3\n    - coreutils\n    - procps\n    - openssl\n    - wget\n    - sudo\n    - py3-requests\nsources:\n    - 'https:\/\/gitlab.com\/postmarketOS\/pmaports.git#411746babf59587634d513261e2efcf13803ea6f'\ntasks:\n    - { setup-pmbootstrap: 'cd pmaports\/.sr.ht ; .\/install_pmbootstrap.sh' }\n    - { add-key: 'mkdir ~\/.local\/var\/pmbootstrap\/config_abuild ; cd ~\/.local\/var\/pmbootstrap\/config_abuild\/ ; cp ~\/.secrets\/build@postmarketos.org.priv . ; openssl rsa -in build@postmarketos.org.priv -pubout -out build@postmarketos.org.pub ; cp build@postmarketos.org.pub \/tmp\/pmbootstrap\/pmb\/data\/keys\/ ; echo PACKAGER_PRIVKEY=\"\/home\/pmos\/.abuild\/build@postmarketos.org.priv\" > abuild.conf' }\n    - { build: 'cd pmaports\/.sr.ht ; pmbootstrap --details-to-stdout --aports \/home\/build\/pmaports -mp=\"https:\/\/build.postmarketos.org\/repository\/master\/main\" -t 900 build --force --strict --arch=x86_64 0xffff' }\n    - { submit: 'cd pmaports\/.sr.ht ; python3 submit.py --id 0xffff:0.7:1 package-submit ~\/.local\/var\/pmbootstrap\/packages\/x86_64\/0xffff-*-r*.apk' }\nenvironment:\n    COMMIT: 411746babf59587634d513261e2efcf13803ea6f\n    BRANCH: master\nsecrets:\n    - c59eb8f9-cd65-4bde-adad-cb6c2e4ae75a\n    - 27bee529-aeb8-4241-b57b-db7e7b8c581d\ntriggers:\n    - { action: webhook, condition: always, url: 'https:\/\/build.postmarketos.org\/api\/failure-hook' }\n","note":"Building 0xffff[x86_64] from commit [411746babf59587634d513261e2efcf13803ea6f](https:\/\/gitlab.com\/postmarketOS\/pmaports\/commit\/411746babf59587634d513261e2efcf13803ea6f)","execute":false}

Response:

HTTP/1.1 200 OK
Server: nginx
Date: Thu, 14 Mar 2019 23:52:48 GMT
Content-Type: application/json
Content-Length: 13
Connection: close

{"id": 42277}
Status
RESOLVED FIXED
Submitter
~martijnbraam
Assigned to
No-one
Submitted
4 months ago
Updated
4 months ago
Labels
No labels applied.

~sircmpwn REPORTED FIXED 4 months ago

It's a bit quiet in here.