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 a license-checking job #1326

Merged
merged 3 commits into from
May 25, 2021
Merged

Add a license-checking job #1326

merged 3 commits into from
May 25, 2021

Conversation

skitt
Copy link
Member

@skitt skitt commented May 12, 2021

This uses https://github.com/uw-labs/lichen, which is closer to our
requirements for CNCF validation than
https://github.com/mitchellh/golicense and performs validation using
local information.

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Depends on #1344
Fixes: #801
Signed-off-by: Stephen Kitt [email protected]

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1326/skitt/licensecheck

@skitt
Copy link
Member Author

skitt commented May 12, 2021

The failure is expected, because go-ipify’s license (the “Unlicense”) isn’t in the approved list and it doesn’t have an exception; this is something I assume we’ll need to fix, either by requesting an exception or by using some other tool (which fits in with the discussions around determining public IP addresses anyway).

@skitt skitt requested a review from dfarrell07 May 12, 2021 16:39
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@dfarrell07
Copy link
Member

The tool and PR overall look great, ACK to your self-review -1 for now.

@dfarrell07 dfarrell07 mentioned this pull request May 13, 2021
87 tasks
skitt added a commit to skitt/submariner-operator that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/lighthouse that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/lighthouse that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/submariner-operator that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/submariner-operator that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/lighthouse that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to skitt/submariner-operator that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
tpantelis pushed a commit to submariner-io/lighthouse that referenced this pull request May 17, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
skitt added a commit to submariner-io/submariner-operator that referenced this pull request May 19, 2021
This uses https://github.com/uw-labs/lichen (see
submariner-io/submariner#1326 for context).

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Signed-off-by: Stephen Kitt <[email protected]>
@github-actions
Copy link
Contributor

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

.lichen.yaml Outdated Show resolved Hide resolved
This uses https://github.com/uw-labs/lichen, which is closer to our
requirements for CNCF validation than
https://github.com/mitchellh/golicense and performs validation using
local information.

The references for the Allowlist and the approved exceptions are
described in the lichen configuration file.

Fixes: submariner-io#801
Signed-off-by: Stephen Kitt <[email protected]>
Copy link
Member

@dfarrell07 dfarrell07 left a comment

Choose a reason for hiding this comment

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

Nice! 🥳

@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label May 25, 2021
@skitt skitt enabled auto-merge (rebase) May 25, 2021 13:05
@skitt skitt merged commit ed47389 into submariner-io:devel May 25, 2021
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1326/skitt/licensecheck]

@skitt skitt deleted the licensecheck branch May 25, 2021 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add dependency license scanning linting
5 participants