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.

Sign up for GitHub

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

Jump to bottom

add support to git functions for buildless serverless #1452

Merged
merged 15 commits into from
Feb 20, 2025
Merged

add support to git functions for buildless serverless #1452

merged 15 commits into from
Feb 20, 2025

Conversation

anoipm
Copy link
Contributor

@anoipm anoipm commented Feb 11, 2025 •
edited
Loading

Description

Changes proposed in this pull request:

  • add support to git functions for buildless serverless - now it works only with public git repositories and checkout sources only on pod start (during initialization phase and doesn't get sources during further reconciliations)

Related issue(s)
See also:

@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cla: yes Indicates the PR's author has signed the CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 11, 2025
@anoipm anoipm force-pushed the buildless-git-public branch from 2fcb28a to 46d6efe Compare February 17, 2025 12:28
@anoipm anoipm force-pushed the buildless-git-public branch from b3056b8 to 7d7dd68 Compare February 18, 2025 10:59
@anoipm anoipm changed the title add git fields to function api in buildless serverless add support to git functions for buildless serverless Feb 18, 2025
@anoipm anoipm marked this pull request as ready for review February 18, 2025 11:55
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 18, 2025
@anoipm anoipm enabled auto-merge (squash) February 19, 2025 13:50
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 20, 2025
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Feb 20, 2025
@kyma-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@anoipm anoipm merged commit 4bfb08d into kyma-project:main Feb 20, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cortey Cortey Cortey approved these changes

@pPrecel pPrecel Awaiting requested review from pPrecel pPrecel is a code owner

@kwiatekus kwiatekus Awaiting requested review from kwiatekus kwiatekus is a code owner

@MichalKalke MichalKalke Awaiting requested review from MichalKalke MichalKalke is a code owner

@halamix2 halamix2 Awaiting requested review from halamix2 halamix2 is a code owner

Assignees

@Cortey Cortey

Labels
cla: yes Indicates the PR's author has signed the CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@anoipm @kyma-bot @Cortey @MichalKalke