Skip to content

Conversation

@mike-sul
Copy link
Contributor

If a target is produced by a container CI job then there is no need in having target.custom.origUriApps value in the target since target.custom.uri refers to a CI job that produced it. The origUriApps should be set only by the LmP target producer to refer to the most recent container CI job that produced apps inherited by a given LmP target.

If a target is produced by a container CI job then there is no need in
having `target.custom.origUriApps` value in the target since
`target.custom.uri` refers to a CI job that produced it. The
`origUriApps` should be set only by the LmP target producer to refer to
the most recent container CI job that produced apps inherited by
a given LmP target.

Signed-off-by: Mike Sul <mike.sul@foundries.io>
@mike-sul mike-sul requested a review from detsch November 15, 2024 15:17
Copy link
Member

@detsch detsch left a comment

Choose a reason for hiding this comment

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

LGTM

@mike-sul mike-sul merged commit bddafc0 into master Nov 18, 2024
1 check passed
@mike-sul mike-sul deleted the fix-app-orig-uri branch November 18, 2024 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants