Skip to content
This repository was archived by the owner on Jun 26, 2024. It is now read-only.

Tweak security vulnerabilities #1184

Merged
merged 1 commit into from
Aug 16, 2022
Merged

Tweak security vulnerabilities #1184

merged 1 commit into from
Aug 16, 2022

Conversation

pmacik
Copy link
Contributor

@pmacik pmacik commented Jul 18, 2022

Signed-off-by: Pavel Macík [email protected]

Changes

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Docs
    included if any changes are user facing
  • Tests
    included if any functionality added or changed. For bugfixes please include tests that can catch regressions
  • All acceptance test scenarios included in the PR which verifies a bugfix or a requested feature reported by a non-member are tagged with @external-feedback tag.
  • Follows the commit message standard

@pmacik pmacik force-pushed the tweek-trivy branch 3 times, most recently from 445e6d6 to 7070a21 Compare July 18, 2022 15:03
@pmacik pmacik changed the title WIP: tweak security scan WIP: tweak security vulnerabilities Jul 18, 2022
@pmacik pmacik force-pushed the tweek-trivy branch 7 times, most recently from bc618fd to 1a39aa6 Compare July 20, 2022 12:48
@pmacik pmacik force-pushed the tweek-trivy branch 3 times, most recently from 393c2e3 to 85f7317 Compare July 26, 2022 16:37
@codecov
Copy link

codecov bot commented Jul 26, 2022 •
edited
Loading

Codecov Report

Merging #1184 (7c296fe) into master (c233e02) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1184   +/-   ##
=======================================
  Coverage   57.96%   57.96%           
=======================================
  Files          33       33           
  Lines        2838     2838           
=======================================
  Hits         1645     1645           
  Misses       1032     1032           
  Partials      161      161

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c233e02...7c296fe. Read the comment docs.

@pmacik
Copy link
Contributor Author

pmacik commented Jul 27, 2022

/test 4.12-acceptance

@pmacik pmacik force-pushed the tweek-trivy branch 2 times, most recently from d4bac12 to 3721fdb Compare July 27, 2022 21:35
@pmacik
Copy link
Contributor Author

pmacik commented Jul 27, 2022

/test 4.12-acceptance

@pmacik pmacik force-pushed the tweek-trivy branch 2 times, most recently from 360f220 to a9aaed0 Compare July 28, 2022 08:26
@pmacik
Copy link
Contributor Author

pmacik commented Jul 28, 2022

/retest

1 similar comment
@pmacik
Copy link
Contributor Author

pmacik commented Jul 28, 2022

/retest

@pmacik pmacik marked this pull request as draft August 11, 2022 05:16
@pmacik pmacik marked this pull request as ready for review August 11, 2022 05:19
@pmacik pmacik changed the title WIP: tweak security vulnerabilities Tweak security vulnerabilities Aug 11, 2022
@pmacik
Tweak security scan
7c296fe 
Signed-off-by: Pavel Macík <[email protected]>
sadlerap
sadlerap approved these changes Aug 11, 2022
View reviewed changes
Copy link
Contributor

@sadlerap sadlerap left a comment

Choose a reason for hiding this comment

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

/lgtm
Assuming that the changes underneath vendor/ are just dependency updates.

@openshift-ci openshift-ci bot added the lgtm label Aug 11, 2022
@pmacik
Copy link
Contributor Author

pmacik commented Aug 11, 2022

Assuming that the changes underneath vendor/ are just dependency updates.

yes

@baijum
Copy link
Contributor

baijum commented Aug 16, 2022

/lgtm
/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 16, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baijum

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pmacik
Copy link
Contributor Author

pmacik commented Aug 16, 2022

/retest

@pmacik pmacik mentioned this pull request Aug 16, 2022
Closed
4 tasks
@openshift-merge-robot openshift-merge-robot merged commit 816b648 into redhat-developer:master Aug 16, 2022
@pmacik pmacik deleted the tweek-trivy branch August 18, 2022 12:09
pmacik added a commit to pmacik/service-binding-operator that referenced this pull request Aug 26, 2022
@pmacik
Tweak security scan (redhat-developer#1184)
04f2806 
Signed-off-by: Pavel Macík <[email protected]>

Signed-off-by: Pavel Macík <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dperaza4dustbit dperaza4dustbit Awaiting requested review from dperaza4dustbit

1 more reviewer

@sadlerap sadlerap sadlerap approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@baijum baijum

@sadlerap sadlerap

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pmacik @baijum @sadlerap @openshift-merge-robot