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

[packit] Propose downstream failed for release v4.6.0-rc2 #19232

Closed
packit-as-a-service bot opened this issue Jul 13, 2023 · 3 comments
Closed

[packit] Propose downstream failed for release v4.6.0-rc2 #19232

packit-as-a-service bot opened this issue Jul 13, 2023 · 3 comments
Assignees
@lsm5
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. stale-issue

Comments

@packit-as-a-service
Copy link

Packit failed on creating pull-requests in dist-git (https://src.fedoraproject.org/rpms/podman.git):

dist-git branch error
f37 See https://dashboard.packit.dev/results/propose-downstream/2942
f38 See https://dashboard.packit.dev/results/propose-downstream/2944
rawhide See https://dashboard.packit.dev/results/propose-downstream/2943

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.
@lsm5 lsm5 self-assigned this Jul 13, 2023
@lsm5
Copy link
Member

lsm5 commented Jul 13, 2023

ughhhh I think I will just get rid of that action script. Building manually.

lsm5 added a commit to lsm5/podman that referenced this issue Jul 14, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
24c58a6 
The pre-sync action constantly breaks and is hard to test.
The lines being added by it downstream were only mean to keep Fedora
happy. But given that they provide no tangible benefit as security teams
look at go.mod and go.sum, there's absolutely no point to having the
action run and break constantly.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers#19232

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
lsm5 added a commit to lsm5/podman that referenced this issue Jul 14, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
4e4078d 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@lsm5 lsm5 mentioned this issue Jul 14, 2023
Merged
lsm5 added a commit to lsm5/skopeo that referenced this issue Jul 14, 2023
@lsm5
Packit: remove pre-sync action
a4aa15f 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@lsm5 lsm5 mentioned this issue Jul 14, 2023
Merged
lsm5 added a commit to lsm5/buildah that referenced this issue Jul 14, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
9cc5d5e 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
This was referenced Jul 14, 2023
Merged
Merged
lsm5 added a commit to lsm5/aardvark-dns that referenced this issue Jul 14, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
75a7b92 
It is currently not possible to reliably test the job-specific pre-sync
action until the subsequent upstream release due to limitations in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security issues in libraries,
along with redhat prodsec's own magic for creating security alerts,
there's absolutely no point to having the pre-sync action run and add a
layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
lsm5 added a commit to lsm5/netavark that referenced this issue Jul 14, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
747384d 
It is currently not possible to reliably test the job-specific pre-sync
action until the subsequent upstream release due to limitations in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security issues in libraries,
along with redhat prodsec's own magic for creating security alerts,
there's absolutely no point to having the pre-sync action run and add a
layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@lsm5 lsm5 mentioned this issue Jul 14, 2023
Merged
openshift-cherrypick-robot pushed a commit to openshift-cherrypick-robot/podman that referenced this issue Jul 17, 2023
@lsm5
Packit: remove pre-sync action
d0dee61 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
lsm5 added a commit to lsm5/skopeo that referenced this issue Jul 20, 2023
@lsm5
Packit: remove pre-sync action
882b087 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
(cherry picked from commit a4aa15f)
Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@lsm5 lsm5 mentioned this issue Jul 20, 2023
Merged
lsm5 added a commit to lsm5/buildah that referenced this issue Aug 9, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
3a15cfd 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
(cherry picked from commit 9cc5d5e)
Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
lsm5 added a commit to lsm5/buildah that referenced this issue Aug 9, 2023
@lsm5
[CI:BUILD] Packit: remove pre-sync action
95e1dfb 
The pre-sync action constantly breaks and is currently not possible to
reliably test until the subsequent upstream release due to limitations
in packit.

The lines being added by the action script to the downstream Fedora spec
were only meant to keep Fedora happy. But given that they provide
no tangible benefit as github notifies us of security
issues in libraries mentioned in go.mod and go.sum, along with redhat
prodsec's own magic for creating security alerts, there's absolutely
no point to having the pre-sync action run and add a layer of uncertainty.

This commit removes the pre-sync action and
`rpm/update-spec-provides.sh`.

Ref: containers/podman#19232

[NO NEW TESTS NEEDED]

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
(cherry picked from commit 9cc5d5e)
Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@github-actions
Copy link

A friendly reminder that this issue had no activity for 30 days.

@lsm5
Copy link
Member

lsm5 commented Aug 14, 2023

This has been fixed. Podman v4.6.1 on Fedora was handled by Packit.

@lsm5 lsm5 closed this as completed Aug 14, 2023
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Nov 13, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lsm5 lsm5

Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. stale-issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lsm5