Skip to content

Conversation

OlivierCazade
Copy link
Collaborator

Description

Fixing multiarch build.
Addressing multiple konflux error:
- the build not being hermetic
- removing deprecated task in the build
- using base image from registry.redhat.io

@OlivierCazade OlivierCazade requested a review from jotak September 13, 2024 15:36
@OlivierCazade OlivierCazade force-pushed the konflux-violations branch 2 times, most recently from be1cea0 to a78cc16 Compare September 13, 2024 15:59
Copy link

codecov bot commented Sep 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.01%. Comparing base (576ec0f) to head (fed9beb).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   56.01%   56.01%           
=======================================
  Files         193      193           
  Lines        9479     9479           
  Branches     1213     1213           
=======================================
  Hits         5310     5310           
  Misses       3792     3792           
  Partials      377      377           
Flag Coverage Δ
uitests 57.72% <ø> (ø)
unittests 51.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@OlivierCazade OlivierCazade force-pushed the konflux-violations branch 2 times, most recently from bb838aa to 36e0bc8 Compare September 13, 2024 18:43
@openshift-ci openshift-ci bot added the lgtm label Sep 16, 2024
@openshift-ci openshift-ci bot removed the lgtm label Sep 16, 2024
@OlivierCazade OlivierCazade force-pushed the konflux-violations branch 19 times, most recently from 4e8eca3 to 1aea045 Compare September 20, 2024 12:15
@OlivierCazade OlivierCazade force-pushed the konflux-violations branch 8 times, most recently from a76ba3c to f833039 Compare September 20, 2024 16:09
@jotak
Copy link
Member

jotak commented Sep 30, 2024

/lgtm
@OlivierCazade I guess we can merge?

@openshift-ci openshift-ci bot added the lgtm label Sep 30, 2024
@OlivierCazade
Copy link
Collaborator Author

/approve

Copy link

openshift-ci bot commented Sep 30, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: OlivierCazade

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

@openshift-merge-bot openshift-merge-bot bot merged commit 760d4ae into netobserv:main Sep 30, 2024
12 checks passed
OlivierCazade added a commit to OlivierCazade/network-observability-console-plugin that referenced this pull request Sep 30, 2024
openshift-merge-bot bot pushed a commit that referenced this pull request Nov 7, 2024
…podman 5.2 … (#607)

* Addressing konflux error and fixing multi arch build with podman 5.2 (#595)

* Adding multiarch support in konflux build and some refactoring (#622)
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.

2 participants