Skip to content
This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Add support for an environ (V2) endpoint #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mach6
Copy link

@mach6 mach6 commented Mar 10, 2022

Add the ability to configure a
DRONE_ENV_PLUGIN_ENDPOINT with the runner

Bump drone-go to v1.6.0
Bump runner-go to 1.7.0

tidy via go mod tidy


Open proposal for this change: drone/proposal#22


Tested on linux-amd64. All changes were modeled after the current code in drone-runner-docker.


I could use some help verifying the changes on these platforms, which I do not have access to.

  • linux-arm
  • linux-arm64
  • linux-386
  • windows-amd64
  • windows-386
  • darwin-amd64
  • freebsd-amd64
  • freebsd-arm
  • freebsd-386
  • netbsd-amd64
  • netbsd-arm
  • openbsd-amd64
  • openbsd-arm
  • openbsd-386
  • dragonfly-amd64
  • solaris-amd64

@CLAassistant
Copy link

CLAassistant commented Mar 10, 2022

CLA assistant check
All committers have signed the CLA.

@hwittenborn
Copy link

Is there a status on this? I just set up this runner for one of my deployments thinking DRONE_ENV_PLUGIN_ENDPOINT would work just fine, as the docs were leading me to believe it did.

hwittenborn added a commit to makedeb/mprweb that referenced this pull request Mar 30, 2022
Looks like Drone Exec runners don't support variables from environment extensions (from which we relied upon). See drone-runners/drone-runner-exec#5.
hwittenborn added a commit to makedeb/mprweb that referenced this pull request Mar 30, 2022
Looks like Drone Exec runners don't support variables from environment extensions (from which we relied upon). See drone-runners/drone-runner-exec#5.
@hwittenborn
Copy link

hwittenborn commented Apr 30, 2022

@bradrydzewski @mach6: Any status on this? Not sure what needs to happen for this to get merged

@mach6
Copy link
Author

mach6 commented May 4, 2022

@bradrydzewski @mach6: Any status on this? Not sure what needs to happen for this to get merged

Hi @hwittenborn - I'm waiting to hear back also

@mach6 mach6 force-pushed the mach6_env_extension branch from a609406 to d3ad422 Compare July 5, 2022 21:47
Add the ability to configure a
DRONE_ENV_PLUGIN_ENDPOINT with the runner

Bump drone-go to v1.6.0
Bump runner-go to 1.7.0

tidy via `go mod tidy`
@mach6 mach6 force-pushed the mach6_env_extension branch from d3ad422 to a486f32 Compare July 5, 2022 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants