cooperspencer / gickup

https://cooperspencer.github.io/gickup-documentation/
Apache License 2.0
940 stars 33 forks source link

Latest docker image (0.10.27) causes infinite config reload #210

Closed miped closed 4 months ago

miped commented 4 months ago

Just filled up my disk with logging. It logs very fast as you can see from the timestamps:

image

Reverting to 0.10.26 fixed it. I think i've got a pretty standard setup:

- name: Add gickup container
  community.docker.docker_container:
    container_default_behavior: compatibility
    name: gickup
    image: buddyspencer/gickup:latest
    pull: true
    networks:
      - name: development
    purge_networks: true
    networks_cli_compatible: false
    volumes:
      - "{{ gickup_data_directory }}:/data:rw"
      - "{{ gickup_ssh_directory }}:/.ssh/:rw"
    env:
      TZ: "{{ nas_timezone }}"
    command: /data/conf.yml
    restart_policy: unless-stopped
    memory: 1g
cooperspencer commented 4 months ago

Can you share your config with me? Just redact the tokens. I had the issue in my tests but I thought I fixed it...

miped commented 4 months ago

Can you share your config with me? Just redact the tokens. I had the issue in my tests but I thought I fixed it...

Here you go:

---
cron: 0 1 * * *  # 1am daily

source:
  github:
    - token: "{{ gickup_github_token }}"
      ssh: true
      sshkey: "{{ gickup_ssh_key }}"

destination:
  gitea:
    - token: "{{ gickup_gitea_token }}"
      createorg: true
      url: "http://gitea:3000/"
      mirrorinterval: 8h

---

source:
  github:
    - token: "{{ gickup_github_token }}"
      ssh: true
      sshkey: "{{ gickup_ssh_key }}"

destination:
  local:
    - path: /data/local/
      structured: true

metrics:
  heartbeat:
    urls:
      - "{{ gickup_healthcheck }}"
cooperspencer commented 4 months ago

Thanks. I'll try to work it out asap.

cooperspencer commented 4 months ago

I found the issue. I am building a fix as we speak. can you test if the issue still occurs with buddyspencer/gickup:cron_fix

miped commented 4 months ago

I found the issue. I am building a fix as we speak. can you test if the issue still occurs with buddyspencer/gickup:cron_fix

Yes, i'll try it out later today

miped commented 4 months ago

@cooperspencer That seems to fix it!

cooperspencer commented 4 months ago

Awesome, thanks for finding the issue. I merged it and will create a new release in a bit.

cooperspencer commented 4 months ago

The new release is out. I'll close the ticket for now. Please reopen it if there are still issues