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

Publishing of 2.4.0 binaries fails #785

Closed
kelson42 opened this issue Dec 15, 2024 · 18 comments · Fixed by #786 or #793
Closed

Publishing of 2.4.0 binaries fails #785

kelson42 opened this issue Dec 15, 2024 · 18 comments · Fixed by #786 or #793
Assignees
Labels

Comments

@kelson42
Copy link
Contributor

https://github.com/kiwix/kiwix-build/actions/runs/12331239742

@kelson42
Copy link
Contributor Author

@veloman-yunkan Really strange that CI works, even CD for nightlies... but not release!

@veloman-yunkan
Copy link
Collaborator

veloman-yunkan commented Dec 16, 2024

There is a suspicious line in the failure log of the linux build:

current directory is '/home/runner/BUILD_linux-x86_64/kiwix-desktop-2.3.1'

@veloman-yunkan
Copy link
Collaborator

Indeed, main_project_versions has not been updated in PR#784.

@veloman-yunkan
Copy link
Collaborator

veloman-yunkan commented Dec 16, 2024

Indeed, main_project_versions has not been updated in PR#784.

That probably explains the build failures both in https://github.com/kiwix/kiwix-build/actions/runs/12331239742 (this ticket) and https://github.com/kiwix/kiwix-desktop/actions/runs/12331074751 (kiwix/kiwix-desktop#1285) - trying to make a new major release of kiwix-desktop using the source code of the old release is not a good idea.

@veloman-yunkan
Copy link
Collaborator

veloman-yunkan commented Dec 16, 2024

Though looking again at the error in kiwix/kiwix-desktop#1285 I have to admit that there may be a different problem there.

@kelson42
Copy link
Contributor Author

kelson42 commented Dec 16, 2024

@veloman-yunkan Looks better now after a new publishing attempt, unfortunqtely we have still errors on Linux, see https://github.com/kiwix/kiwix-build/actions/runs/12360372427:

  • The flatpak repository upload fails, it seems we have the wrong credentials... might be a problem we already had, can not say for the moment.
  • On linux job fails, not sure what was the output... pu whatbisnsure is that the source tarball is notmvisible at http://download.kiwix.org/release/kiwix-desktop/?C=M;O=D
  • In addition the windows ZIP file has somehow a bad filename (is kiwix-desktop_windows_x64_2.4.zip but should be ``kiwix-desktop_windows_x64_2.4.0.zip`)

@kelson42 kelson42 reopened this Dec 16, 2024
@kelson42
Copy link
Contributor Author

  • In addition the windows ZIP file has somehow a bad filename (is kiwix-desktop_windows_x64_2.4.zip but should be ``kiwix-desktop_windows_x64_2.4.0.zip`)

I believe to have fixed this point with b1b545a

@veloman-yunkan
Copy link
Collaborator

Should be fixed by #787

@veloman-yunkan
Copy link
Collaborator

  • The flatpak repository upload fails, it seems we have the wrong credentials... might be a problem we already had, can not say for the moment.

This is currently outside my reach.

@veloman-yunkan
Copy link
Collaborator

  • In addition the windows ZIP file has somehow a bad filename (is kiwix-desktop_windows_x64_2.4.zip but should be ``kiwix-desktop_windows_x64_2.4.0.zip`)

I believe to have fixed this point with b1b545a

The fix was wrong:

--- creating desktop image ---
Traceback (most recent call last):
  File "D:\a\kiwix-build\kiwix-build\.github\scripts\build_projects.py", line 18, in <module>
    archive = create_desktop_image(make_release=False)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\a\kiwix-build\kiwix-build\.github\scripts\common.py", line 547, in create_desktop_image
    build_path = working_dir + ".zip"
                 ~~~~~~~~~~~~^~~~~~~~
TypeError: unsupported operand type(s) for +: 'WindowsPath' and 'str'
Error: Process completed with exit code 1.

I corrected it with the second commit in #787.

@kelson42
Copy link
Contributor Author

kelson42 commented Dec 18, 2024

  • The flatpak repository upload fails, it seems we have the wrong credentials... might be a problem we already had, can not say for the moment.

This is currently outside my reach.

The idea would be to upload this via KiwixBot, this has been long dine that way, but for some reason has stopped to work a few releases ago. We shoukd not rely on someone doing this manually... we see right now the problem of this approach. Do you have some kind of credential issue?

The general approach would be to fix this and the redo a Kiwix desktop patch release 2.4.1

@veloman-yunkan
Copy link
Collaborator

Do you have some kind of credential issue?

I don't think I have such an issue. So far I simply haven't dealt with Kiwix organization's credentials in any way (and I believe that I have no access to those).

@kelson42
Copy link
Contributor Author

@veloman-yunkan They are in the GitHub secrets. Do you want to have them to test locally or is the current situation good enough?

@veloman-yunkan
Copy link
Collaborator

They are in the GitHub secrets.

@kelson42 I don't have access to GitHub secrets (or any settings at all) for this repository.

@kelson42
Copy link
Contributor Author

They are in the GitHub secrets.

@kelson42 I don't have access to GitHub secrets (or any settings at all) for this repository.

You are now admin of this repository... but secrets been secrets you can only modify them, not read them in GitHub.

@kelson42
Copy link
Contributor Author

Regarding this last Flathub update:

@veloman-yunkan I will send you the credentials and hope you can have a look. Maybe you can try manuall to make 2.4.0 from the script and if OK we can make a full release of 2.4.1.

@kelson42
Copy link
Contributor Author

The problem of credentials has been solved by @rgaudin, but now we have a problem with compilation, see https://buildbot.flathub.org/#/builders/9/builds/15859

@kelson42
Copy link
Contributor Author

@veloman-yunkan Finalement I believe #793 fixes the publishing on Flathub... so I will close this issue and prepare 2.4.1 release to verify everything works fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants