cschlosser / drone-ftps

Deploy to FTPS server from Drone CI build
https://hub.docker.com/r/cschlosser/drone-ftps/
Apache License 2.0
20 stars 16 forks source link

Stuck on master_build step #23

Closed morvy closed 2 years ago

morvy commented 2 years ago

Hi,

sorry for opening an issue but I am trying to get drone to upload pushed commit to FTP with no luck.

My .drone.yml looks like this:

kind: pipeline
name: default

steps:
- name: master_build
  image: cschlosser/drone-ftps
  environment:
    FTP_USERNAME:
      from_secret: username
    FTP_PASSWORD:
      from_secret: password
    PLUGIN_HOSTNAME: mytestftp.com:21
    PLUGIN_SECURE: false
    PLUGIN_VERIFY: false
    PLUGIN_EXCLUDE: ^\.git/$

..so it's 1:1 to the example here, secrets are set under username and password in drone repository secrets. I've also tried config I found on drone plugins site, with no luck too. If I add when clause, it won't trigger at all

In log I'm stuck on this message:

latest: Pulling from cschlosser/drone-ftps
Digest: sha256:7442ec050d7b05c3ff4f14e92e9bb0def4028ba555844591815434de8a5fe8ee
Status: Image is up to date for cschlosser/drone-ftps:latest

Any idea what I'm doing wrong? Thanks!

karlludwigweise commented 2 years ago

I think you are missing PLUGIN_SRC_DIR env var ;)

karlludwigweise commented 2 years ago

Just saw its documented as optional. But it's the only other var I declare in a working example.

morvy commented 2 years ago

Thanks for checking, I will try to add it and see if it helps. I left the build running and this morning I found files on FTP, but it took 57 minutes to start the copy process, which is odd.

cschlosser commented 2 years ago

Hi @morvy ,

don't worry, happy to help (and thank you @karlludwigweise for chiming in already).

The config looks good and the optional settings should truly be optional.

Could you try and start the image without drone on the machine and see what happens?

morvy commented 2 years ago

Hi, a bit late but I didn't have an opportunity to try Drone again in a while. I tried the same build again and it seems to be working fine.

cschlosser commented 2 years ago

Thanks for reporting back!