Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix temp path #112

Closed
wants to merge 2 commits into from
Closed

Fix temp path #112

wants to merge 2 commits into from

Conversation

lukeb2e
Copy link

@lukeb2e lukeb2e commented Mar 8, 2018

Fixes issue in #110

@mkloveyy
Copy link

Why don't merge this branch into master?

@mkloveyy
Copy link

Sry, I build it on my own but it still not works.

@gojihotsauce
Copy link

I can confirm that the issue still exists, even when building from this branch.

@mkloveyy
Copy link

mkloveyy commented Jun 23, 2018

@debtcollapse
Recently, I tried to use docker-compose way to run clair again. I adjusted something in docker-compose.yml and it works. Here is my docker-compose.yml and I hope it may be useful to you:

services:
  postgres:
    container_name: clair_postgres
    image: postgres:latest
    restart: unless-stopped
    environment:
      POSTGRES_PASSWORD: password

  clair:
    container_name: clair_clair
    #image: quay.io/coreos/clair-git:latest
    image: quay.io/coreos/clair:latest
    restart: unless-stopped
    depends_on:
      - postgres
    ports:
      - "6060-6061:6060-6061"
    links:
      - postgres
    volumes:
      - /tmp:/tmp
      - ./clair_config:/config
    command: [-config, /config/config.yaml]

  registry:
    container_name: registry
    image: registry:2
    restart: unless-stopped
    ports:
      - "5000:5000"

  clairctl:
    image: jgsqware/clairctl:latest
    container_name: clair_clairctl
    restart: unless-stopped
    environment:
      - DOCKER_API_VERSION=1.24
    volumes:
      - ./docker-compose-data/clairctl-reports/:/reports/:rw
      - /var/run/docker.sock:/var/run/docker.sock:ro
    user: root
    group_add:
        - root
    depends_on:
      clair:
        condition: service_started

@khalilj
Copy link

khalilj commented Aug 22, 2018

@mkloveyy what are the exact changes? adding registry to compose?

@LucLIO
Copy link

LucLIO commented Dec 6, 2018

Let's give this a little kick :)
Any news on this front?
I have read like 40 issues, tried about everything from a local registry to the many group_add and user modifications in the docker-compose file. Still have some images from our own repo and from Docker Hub that end in error 400 when pushing the first layer to clair.

@LucLIO
Copy link

LucLIO commented Dec 7, 2018

For the record, I tried with the private repo with the same images that were giving the error 400 when pushing the first layer. First I thought it was our Nexus server that was giving issues. But I found out it was also with some from the Docker Hub. The route cause seems indeed to be in the image itself or the process. The image that failed for me from Docker Hub is the monsantoco/logstash.
Hope you guys can help out as I'm building this as a proof of concept in my final internship before I graduate :)

@lukeb2e lukeb2e closed this Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants