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

Image building and pushing for free5gc-operator #27

Merged
merged 3 commits into from
May 5, 2023

Conversation

radoslawc
Copy link
Collaborator

Images built from every commit will have prow BUILD_ID as a version and those build from release tags v.X.Y.Z will have that tag name as version

Images built from every commit will have prow BUILD_ID as a version and those build from release tags v.X.Y.Z will have that tag name as version
prow/config/config.yaml Show resolved Hide resolved
prow/config/config.yaml Outdated Show resolved Hide resolved
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
Copy link

@gvbalaji gvbalaji left a comment

Choose a reason for hiding this comment

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

/approve

@nephio-prow
Copy link
Contributor

nephio-prow bot commented May 5, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gvbalaji, radoslawc

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

@electrocucaracha
Copy link
Member

/lgtm

@nephio-prow nephio-prow bot added the lgtm label May 5, 2023
@nephio-prow nephio-prow bot merged commit c58c18f into main May 5, 2023
@nephio-prow
Copy link
Contributor

nephio-prow bot commented May 5, 2023

@radoslawc: Updated the config configmap in namespace prow at cluster default using the following files:

  • key config.yaml using file prow/config/config.yaml

In response to this:

Images built from every commit will have prow BUILD_ID as a version and those build from release tags v.X.Y.Z will have that tag name as version

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@radoslawc radoslawc deleted the radoslawc-patch-4 branch May 8, 2023 15:30
@radoslawc
Copy link
Collaborator Author

That resolves nephio-project/nephio#160 issue, for other repos when Dockerfile will be provided the same pipeline will be applied.

resources:
requests:
cpu: 2
memory: 2Gi
Copy link

Choose a reason for hiding this comment

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

No a problem / blocker. But 2 vCPUs and 2 GB memory seems like a LOT for the operator...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants