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

Add nancy dependency checking #43

Merged
merged 3 commits into from
Oct 14, 2020

Conversation

deadlysyn
Copy link
Contributor

Updated CI config to dogfood nancy.

This pull request makes the following changes:

  • Added dependency checking to circleci config
  • Workaround etcd vuln
  • Bump dependencies

It relates to the following issue #s:

cc @bhamail / @DarthHater

Copy link
Contributor

@zendern zendern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

I see the nancy run in circleCI. 👏 @deadlysyn

@deadlysyn
Copy link
Contributor Author

FYI it looks like they changed hacktoberfest requirements...good intent of preventing spam, but more work to participate. More proof spam ruins lives. 😞

https://hacktoberfest.digitalocean.com/hacktoberfest-update

Copy link
Contributor

@bhamail bhamail left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 I'm gonna merge this puppy shortly.

Just as a heads-up, I saw the following error when trying to run a local CircleCI build. I'm guessing it is some sort of macos/docker thang:

    docker_test.go:47: >>>>>>>>>>>>>> yum using autodetect: yum-autodetect/Dockerfile
Error: exit status 1
Output 1: 1
=== CONT  TestDockerIntegration/docker/dnf
    docker_test.go:49: >>>>>>>>>>>>>> dnf: Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.40/build?buildargs=%7B%7D&cachefrom=%5B%5D&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=dnf%2FDockerfile&labels=%7B%7D&memory=0&memswap=0&networkmode=default&nocache=1&rm=1&session=sh3zhq9b7r48s0fermd2mbok6&shmsize=0&target=&ulimits=null&version=1: dial unix /var/run/docker.sock: connect: permission denied

I also thought I saw some similar related discussion elsewhere. Just mentioning in case you need a labrat to try stuff on macos - I can try stuff out as needed.

@bhamail bhamail merged commit 8182726 into sonatype-nexus-community:master Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dogfood Nancy
3 participants