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

chore: Fix Build Image tags #1970

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

c-pius
Copy link
Contributor

@c-pius c-pius commented Oct 18, 2024

Description

Changes proposed in this pull request:

  • fixes the provided custom tags

pull_request_target makes this PR use the old image build func here. Actual test is here: #1971

Related issue(s)

@c-pius c-pius requested a review from a team as a code owner October 18, 2024 10:13
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 18, 2024
@c-pius c-pius force-pushed the chore/fix-image-build-tags branch from 2d376cb to 1d748c2 Compare October 18, 2024 10:48
@kyma-bot kyma-bot added the lgtm Looks good to me! label Oct 18, 2024
@c-pius c-pius enabled auto-merge (squash) October 18, 2024 11:08
@c-pius c-pius merged commit 95297df into kyma-project:main Oct 18, 2024
61 checks passed
@c-pius c-pius deleted the chore/fix-image-build-tags branch October 18, 2024 11:11
c-pius added a commit to kyma-project/template-operator that referenced this pull request Oct 21, 2024
<!-- Thank you for your contribution. Before you submit the pull
request:
1. Follow contributing guidelines, templates, the recommended Git
workflow, and any related documentation.
2. Read and submit the required Contributor Licence Agreements
(https://github.com/kyma-project/community/blob/main/CONTRIBUTING.md#agreements-and-licenses).
3. Test your changes and attach their results to the pull request.
4. Update the relevant documentation.
-->

**Description**

Changes proposed in this pull request:

- fixes the provided custom tags

`pull_request_target` makes this PR use the old image build func here.
Didn't test separately again because it has been tested as part of:
kyma-project/lifecycle-manager#1970

**Related issue(s)**
<!-- If you refer to a particular issue, provide its number. For
example, `Resolves #123`, `Fixes #43`, or `See also #33`. -->

- partially closes
https://github.com/orgs/kyma-project/projects/30/views/13?pane=issue&itemId=83441848
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants