-
Notifications
You must be signed in to change notification settings - Fork 42
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
New release #257
Comments
#255 merge then seems good for a new point release. Is there a consensus change? I do not recall anything that would require this being added since prior point release. If not then looks good for a point release. It has been long enough and lots of enhancements plus compile fixes. |
I agree that a new point release is indeed desired now because of the stale inactive seed nodes issue (eg https://www.reddit.com/r/Aeon/comments/r24ubc/comment/hm54144/). |
Are the upstream PRs ready to merge (both CLI and GUI)? |
Should work well with new seed nodes. |
Bump
|
Sorry for my slow response, been very busy these days. They should be ready, but please check by yourself if you can. |
Reviewed |
Looks like GUI has one more PR plus a version bump. aeonix/aeon-gui#58 Same as CLI |
This looks close to ready. There are a couple of MacOS PRs that need review. They look small, but I'm not familiar enough with what they are doing to review myself. After that we will need version bump and binaries built. |
Recommend version bump commit and release name poll on social media. #261 can stay open for the time being. Same for gui pr. |
There's no need for release name poll because this is going to be just a point release |
@stoffu Are you able to do a version bump commit and also please explain how the new process for binaries works. Thanks. |
Currently I’m struggling with annoying issues in the GUI (crashes when opening the Settings tab in Windows & Mac, not launching at all on Linux). I’m configuring the GitHub Actions CI such that a custom built Qt can be used. When completed, obtaining release binaries should be as simple as just downloading the built artifacts from the CI. For example, you can get the current master binaries from https://github.com/aeonix/aeon-gui/actions/runs/1799574070#artifacts (which don't run correctly due to the Qt issues). The issue is relevant for the GUI only, so we could make new release for the CLI first, and do the GUI release later when the Qt issue gets solved. Should we go this way, or is it better to release the CLI & GUI at the same time? |
Let's wait a little while and see if you can make any progress with the GUI. If not we can push out the CLI separately. |
Finally I think I figured out the right way of producing release binaries. With these patches #264 & aeonix/aeon-gui#63, the following binaries have been generated by GitHub Actions CI (the corresponding CI runs are https://github.com/aeonix/aeon/actions/runs/1896600423 & https://github.com/stoffu/monero-core/actions/runs/1916874316/attempts/6, respectively): Please download the following release candidate binaries and report if you see any issues:
SHA256 hashes:
Some explanations
|
Do the above binaries have all of the expected changes set for v0.14.2.0 (so far) and are pretty much pre-release binaries just versioned v0.14.1.0? |
@BigslimVdub Yes. |
Ready for version bump PR? |
I’m a bit reluctant to do that since currently there does seem to exist an issue of aeond occasionally getting stuck while in initial blockchain download, as was complained in #265 and confirmed in #267. I’m currently trying to figure out what’s going on, but my investigation is slow due to my other life duties. I hope to get this fixed in a few weeks or so. Or should we rush the release now anyway ignoring this issue? |
Honestly I’ve had intermittent initial chain download issues since 2016 but they have been a lot less since rebase and it may also be related to outdated seed nodes since closing daemon and restarting usually resolves the issue if it’s stuck or can’t start initial sync. Are you still seeing issues with a current master pull with the updated seed node addresses? I had no issues on OS X m1 and Ubuntu 20.04 with full downloads from 0. I did not try windows or intel OS X. |
Yes, I did observe this IBD stuck issue on macOS and Ubuntu. This can be confirmed also by running the GitHub CI of #267 yourself. |
Seems fine to continue working on the problem unless unable to make progress in which case it may be worthwhile to get the other improvements out. |
The IBD test in #267 now passes, so I think we can move on to releasing. |
Is there a way to get CI to upload binaries to the release automatically or does it need to be done manually? |
I quickly searched the web and found this https://github.com/softprops/action-gh-release, so I'm trying it out in #272. |
No sign of binaries. I guess it didn't work. |
It did work. All the binaries have been uploaded there. |
Indeed. I guess there is some processing delay. |
Can we upload the GUI binaries in a similar manner? |
Yes, once merged aeonix/aeon-gui#64 and created a release with tag |
Does it make sense to sign hashes on these? Someone asked on reddit. I'm not sure personally whether it makes sense but I guess it guards against the repo being compromised and the binaries silently replaced? |
I plan to sign on the hashes. Just I haven't checked if they all run fine. Will put the signature here soon. |
Is this a good point for a new release? There seem to be a lot of upstream merges as well as the much needed update to the seed nodes.
Or is there some ongoing work that would make it better to wait?
The text was updated successfully, but these errors were encountered: