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

feat: Make container build engine configurable #87

Merged
merged 4 commits into from
Sep 19, 2024
Merged

Conversation

spolti
Copy link
Contributor

@spolti spolti commented May 23, 2024

chore: Allow to run make goals provinding the desired Container Enginer builder tool,
e.g. ENGINE=podman make build

Motivation

Modifications

Result

chore:	Allow to run make goals provinding the desired Container Enginer builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>
@oss-prow-bot oss-prow-bot bot requested review from ckadner and tjohnson31415 May 23, 2024 14:28
@rafvasq rafvasq changed the title Make contaier buld engine configurable feat: Make container build engine configurable May 27, 2024
Copy link
Member

@ckadner ckadner left a comment

Choose a reason for hiding this comment

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

Looks good. Feels like Déjà vu, though. Did we not already do this a few month ago? 🤔

Copy link

oss-prow-bot bot commented May 27, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ckadner, spolti

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

@spolti
Copy link
Contributor Author

spolti commented Jul 1, 2024

@ckadner do you think it can be merged?

@spolti
Copy link
Contributor Author

spolti commented Sep 19, 2024

Hi @ckadner, do you think we can merge it?

cc @rafvasq

Copy link
Member

@rafvasq rafvasq left a comment

Choose a reason for hiding this comment

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

/lgtm

@rafvasq rafvasq merged commit 4aaa21f into kserve:main Sep 19, 2024
6 of 7 checks passed
@spolti
Copy link
Contributor Author

spolti commented Sep 19, 2024

Thanks.

@spolti spolti deleted the podman branch September 19, 2024 13:38
openshift-merge-bot bot referenced this pull request in opendatahub-io/modelmesh-runtime-adapter Oct 11, 2024
chore: Allow to run make goals provinding the desired Container Enginer
builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>
openshift-merge-bot bot referenced this pull request in opendatahub-io/modelmesh-runtime-adapter Nov 20, 2024
* feat: Make container build engine configurable (#87)

chore: Allow to run make goals provinding the desired Container Enginer
builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies

chore: Update dependencies

Signed-off-by: Spolti <[email protected]>

---------

Signed-off-by: Spolti <[email protected]>
openshift-merge-bot bot referenced this pull request in opendatahub-io/modelmesh-runtime-adapter Jan 23, 2025
* feat: Make container build engine configurable (#87)

chore: Allow to run make goals provinding the desired Container Enginer
builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies

chore: Update dependencies

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies  (kserve#97)

chore:  Fixes the following CVEs:
CVE-2023-45288 - Non-linear parsing of case-insensitive content in
golang.org/x/net/html
CVE-2024-45337 - Misuse of ServerConfig.PublicKeyCallback may cause
authorization bypass in golang.org/x/crypto

#### Motivation

#### Modifications

#### Result

Signed-off-by: Spolti <[email protected]>

---------

Signed-off-by: Spolti <[email protected]>
openshift-merge-bot bot referenced this pull request in opendatahub-io/modelmesh-runtime-adapter Feb 20, 2025
* feat: Make container build engine configurable (#87)

chore: Allow to run make goals provinding the desired Container Enginer
builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies

chore: Update dependencies

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies  (kserve#97)

chore:  Fixes the following CVEs:
CVE-2023-45288 - Non-linear parsing of case-insensitive content in
golang.org/x/net/html
CVE-2024-45337 - Misuse of ServerConfig.PublicKeyCallback may cause
authorization bypass in golang.org/x/crypto

#### Motivation

#### Modifications

#### Result

Signed-off-by: Spolti <[email protected]>

---------

Signed-off-by: Spolti <[email protected]>
Signed-off-by: Brett Thompson <[email protected]>
Co-authored-by: Filippe Spolti <[email protected]>
openshift-merge-bot bot referenced this pull request in opendatahub-io/modelmesh-runtime-adapter Feb 26, 2025
* feat: Make container build engine configurable (#87)

chore: Allow to run make goals provinding the desired Container Enginer
builder tool,
	e.g. `ENGINE=podman make build`

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies

chore: Update dependencies

Signed-off-by: Spolti <[email protected]>

* chore: Update dependencies  (kserve#97)

chore:  Fixes the following CVEs:
CVE-2023-45288 - Non-linear parsing of case-insensitive content in
golang.org/x/net/html
CVE-2024-45337 - Misuse of ServerConfig.PublicKeyCallback may cause
authorization bypass in golang.org/x/crypto

#### Motivation

#### Modifications

#### Result

Signed-off-by: Spolti <[email protected]>

* Upgrade python to 3.11 (kserve#98)

chore:	Update python to 3.11 in preparation of UBI9 upgrade.

#### Motivation

#### Modifications

#### Result

Signed-off-by: Spolti <[email protected]>

---------

Signed-off-by: Spolti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants