Upgrade Drone build file format from 0.8 to 1.x. #187

Merged
witten merged 10 commits from upgrade-drone into master 2019-05-21 02:21:46 +00:00
Owner

Drone 1.x has a completely new file format, and explicit build matrix support is gone.

Drone 1.x has a completely new file format, and explicit build matrix support is gone.
witten closed this pull request 2019-05-21 02:21:46 +00:00
witten deleted branch upgrade-drone 2019-05-21 02:22:29 +00:00
Contributor

explicit build matrix support is gone.

Is Drone the project doing weird things like this a lot!?

> explicit build matrix support is gone. Is Drone the project doing weird things like this a lot!?
Author
Owner

Dunno.. This is the first major Drone upgrade I've been through. Here's more info on the build matrix deprecation: https://docs.drone.io/user-guide/pipeline/migrating/#matrix

As you can see there, they do have a replacement matrix-like feature using jsonnet templating. I started going down that road, only to realize that it doesn't work out of the box and requires running a separate extension to make it work. We could certainly do that in the future, but I didn't want to bite that off as part of the upgrade.

And the related issue where most of the upgrade work was done: witten/provisioning#21

Dunno.. This is the first major Drone upgrade I've been through. Here's more info on the build matrix deprecation: https://docs.drone.io/user-guide/pipeline/migrating/#matrix As you can see there, they do have a replacement matrix-like feature using jsonnet templating. I started going down that road, only to realize that it doesn't work out of the box and requires running a [separate extension](https://docs.drone.io/extend/config/jsonnet/) to make it work. We could certainly do that in the future, but I didn't want to bite that off as part of the upgrade. And the related issue where most of the upgrade work was done: https://projects.torsion.org/witten/provisioning/issues/21
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#187
No description provided.