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 v1.16.0 #273

Closed
packit-as-a-service bot opened this issue Sep 19, 2023 · 17 comments
Closed

[packit] Propose downstream failed for release v1.16.0 #273

packit-as-a-service bot opened this issue Sep 19, 2023 · 17 comments
Assignees
@dav-pascual
Labels
bug Something isn't working CI/CD Label to use for automation changes in CI/CD release Release candidate PR

Comments

@packit-as-a-service
Copy link

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

dist-git branch error
epel8 See https://dashboard.packit.dev/results/propose-downstream/3901
epel9 See https://dashboard.packit.dev/results/propose-downstream/3905
f37 See https://dashboard.packit.dev/results/propose-downstream/3903
f38 See https://dashboard.packit.dev/results/propose-downstream/3904
f39 See https://dashboard.packit.dev/results/propose-downstream/3902
rawhide See https://dashboard.packit.dev/results/propose-downstream/3906

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.
@packit-as-a-service
Copy link
Author

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

dist-git branch error
epel8 See https://dashboard.packit.dev/results/propose-downstream/3907
epel9 See https://dashboard.packit.dev/results/propose-downstream/3911
f37 See https://dashboard.packit.dev/results/propose-downstream/3909
f38 See https://dashboard.packit.dev/results/propose-downstream/3910
f39 See https://dashboard.packit.dev/results/propose-downstream/3908
rawhide See https://dashboard.packit.dev/results/propose-downstream/3912

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
f38 See https://dashboard.packit.dev/results/propose-downstream/3924

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
epel8 See https://dashboard.packit.dev/results/propose-downstream/3928

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
f39 See https://dashboard.packit.dev/results/propose-downstream/3925

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
epel9 See https://dashboard.packit.dev/results/propose-downstream/3927

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
f37 See https://dashboard.packit.dev/results/propose-downstream/3926

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
rawhide See https://dashboard.packit.dev/results/propose-downstream/3929

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@Tiboris Tiboris added the bug Something isn't working label Sep 25, 2023
@Tiboris
Copy link
Member

Tiboris commented Sep 25, 2023

Could you please have a look @dav-pascual ?

@Tiboris Tiboris added release Release candidate PR CI/CD Label to use for automation changes in CI/CD labels Oct 4, 2023
@Tiboris Tiboris pinned this issue Oct 4, 2023
@dav-pascual dav-pascual mentioned this issue Oct 5, 2023
Merged
dav-pascual added a commit to dav-pascual/mrack that referenced this issue Oct 5, 2023
@dav-pascual
chore(ci): Fix release workflow build step checking out wrong commit
dda3b4f 
actions/checkout checks out the latest commit at the time of the workflow trigger,
hence any commit done during the workflow is not included.
Fixing this to checkout the actual latest (release) commit.

Fixes: neoave#273

Signed-off-by: David Pascual <[email protected]>
dav-pascual added a commit to dav-pascual/mrack that referenced this issue Oct 5, 2023
@dav-pascual
chore(ci): Fix release workflow build step checking out wrong commit
8028246 
actions/checkout checks out the latest commit at the time of the workflow trigger,
hence any commit done during the workflow is not included.
Fixing this to checkout the actual latest (release) commit.

Fixes: neoave#273

Signed-off-by: David Pascual <[email protected]>
@dav-pascual
Copy link
Member

/packit propose-downstream

@packit-as-a-service
Copy link
Author

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

dist-git branch error
epel8 See https://dashboard.packit.dev/results/propose-downstream/4200
epel9 See https://dashboard.packit.dev/results/propose-downstream/4198
f37 See https://dashboard.packit.dev/results/propose-downstream/4196
f38 See https://dashboard.packit.dev/results/propose-downstream/4197
f39 See https://dashboard.packit.dev/results/propose-downstream/4201
rawhide See https://dashboard.packit.dev/results/propose-downstream/4199

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@packit-as-a-service
Copy link
Author

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

dist-git branch error
epel8 See https://dashboard.packit.dev/results/propose-downstream/4204
epel9 See https://dashboard.packit.dev/results/propose-downstream/4202
f37 See https://dashboard.packit.dev/results/propose-downstream/4207
f38 See https://dashboard.packit.dev/results/propose-downstream/4205
f39 See https://dashboard.packit.dev/results/propose-downstream/4203
rawhide See https://dashboard.packit.dev/results/propose-downstream/4206

You can retrigger the update by adding a comment (/packit propose-downstream) into this issue.

@Tiboris
Copy link
Member

Tiboris commented Oct 10, 2023 •
edited
Loading

@dav-pascual https://github.com/svenstaro/upload-release-action add this step after python packages are built and upload them to release assets

I think this might be the start for you

    - name: Upload python packages to GH release assets
      uses: svenstaro/upload-release-action@v2
      with:
        repo_token: ${{ secrets.GITHUB_TOKEN }}
        file: (the packages built)
        asset_name: (match here what packit expects)
        tag: ${{ github.ref }}
        overwrite: true

maybe after this line

mrack/.github/workflows/test-and-release.yml

Line 119 in 8e30c75

run: python3 -m build

@Tiboris
Copy link
Member

Tiboris commented Oct 10, 2023

I would test this outside of this repository in your fork pypi upload now works and you can easily remove release from your repo (for safety remove upload to pypi automation from your fork)

dav-pascual added a commit to dav-pascual/mrack that referenced this issue Oct 11, 2023
@dav-pascual
chore(release): Upload distribution package to release assets
6e46fcc 
To fix fedora release, broken in neoave#272

Fixes neoave#273

Signed-off-by: David Pascual <[email protected]>
dav-pascual added a commit to dav-pascual/mrack that referenced this issue Oct 11, 2023
@dav-pascual
chore(release): Upload distribution package to release assets
49a98bd 
To fix fedora release, broken in neoave#272

Fixes neoave#273

Signed-off-by: David Pascual <[email protected]>
dav-pascual added a commit to dav-pascual/mrack that referenced this issue Oct 11, 2023
@dav-pascual
chore(release): Upload distribution package to release assets
a0f5372 
To fix fedora release, broken in neoave#272

Fixes neoave#273

Signed-off-by: David Pascual <[email protected]>
@dav-pascual
Copy link
Member

This PR should fix this issue #277
Will test after it's merged

@Tiboris Tiboris reopened this Oct 17, 2023
@Tiboris
Copy link
Member

Tiboris commented Oct 17, 2023

/packit propose-downstream

@Tiboris
Copy link
Member

Tiboris commented Oct 17, 2023

@dav-pascual great job PRs are now created https://src.fedoraproject.org/rpms/mrack/pull-requests

@Tiboris
Copy link
Member

Tiboris commented Oct 17, 2023

solved

@Tiboris Tiboris closed this as completed Oct 17, 2023
@dav-pascual dav-pascual unpinned this issue Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dav-pascual dav-pascual

Labels
bug Something isn't working CI/CD Label to use for automation changes in CI/CD release Release candidate PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Tiboris @dav-pascual