rcv icon indicating copy to clipboard operation
rcv copied to clipboard

Fix/Clean Up Development Builds

Open nurse-the-code opened this issue 1 year ago • 12 comments

We recently updated the development builds to specify which CPU architecture is being used. It appears that resulted in two problems:

  1. We have files from multiple fortnightly development builds in the same release section. (see the attached screen shot). After talking with @artoonie, it sounds like when creating a fortnightly build, we should delete the old fortnightly build files.
  2. We stopped generating .sha512 files for the executable installers (.deb, .dmg, .exe) when we started generating the .golden.sha512 for the .zip files. Let's add those back in.
Screenshot 2024-06-28 at 12 37 52

nurse-the-code avatar Jul 01 '24 14:07 nurse-the-code

@artoonie partially fixed the first part by deleting the old build files. It looks like we still have old .zip and .tar.gz files from May 31st.

But the second part still does not appear to be fixed:

We stopped generating .sha512 files for the executable installers (.deb, .dmg, .exe) when we started generating the .golden.sha512 for the .zip files. Let's add those back in.

Finally, while most of the build files say 2 days ago (from the most recent fortnightly build), the header for some reason says "May 31" and the description text says that it is a build of changes "as of June 2nd". Can we fix that too?

Screenshot 2024-07-03 at 14 43 49

nurse-the-code avatar Jul 03 '24 18:07 nurse-the-code

I have fixed both of those in PR #860

Note that there are additional files in the "Package", which is separate from the "Release": https://github.com/BrightSpots/rcv/actions/runs/9744047355

The package is 700mb and contains every cache file for every OS we support. I believe it was an intentional decision to not include these files in the release, since they're only useful for airgapped builds, but cannot be sure. @yezr do you recall why we decided to have some files in the "Package" not included in the "Release"?

Also -- we should document this in the wiki since I forgot about it!

artoonie avatar Jul 03 '24 19:07 artoonie

I cannot remember why we would leave out the airgap caches. I think we were just scrambling to get it out for that certification and overlooked adding it to the official release?

yezr avatar Jul 03 '24 19:07 yezr

Alright, I've added all cache files back in. I personally find this much sloppier, since 99% of users don't need the cache files, and this makes it quite hard to find which files are important. I imagine a better naming scheme could help (e.g. all cache files begin with cache_?). Let me know what you think is needed here: https://github.com/BrightSpots/rcv/releases/tag/build-double-develop

artoonie avatar Jul 03 '24 20:07 artoonie

So the cache files are always created and are part of the build Package? I imagine that for any user who needs an airgapped build that we would be personally involved. In that case, we can always provide them the cache and it wouldn't be required to be a part of the build artifacts.

To make the release a little cleaner, and maybe prevent confusion, I'm fine with not making those an explicit part of the release but leave them in the Package.

yezr avatar Jul 08 '24 17:07 yezr

@artoonie I noticed that we don't have a current development build for arch64 Linux. Does https://github.com/BrightSpots/rcv/pull/860 fix this?

Not sure if we need a build for this platform. I am only asking because my main Linux machine is arch64.

nurse-the-code avatar Jul 15 '24 20:07 nurse-the-code

The linux build should be 64-bit -- does it not work on your linux box?

artoonie avatar Jul 25 '24 18:07 artoonie

The linux build should be 64-bit -- does it not work on your linux box?

My main linux machine is a MacBook Air with linux installed. So it's ARM64/aarch64 rather than x86_64 (e.g. Intel, AMD).

Also, if I were to run Linux virtualized on my MacBook Pro, I would probably want to use an ARM64 build of linux (and RCTab) rather than trying to emulated x86_64.

nurse-the-code avatar Aug 16 '24 21:08 nurse-the-code

Ah, understood, thanks for the clarification -- and yes, I agree, we should build for aarch64.

It looks like it should be a feature coming soon to github actions, but is not a currently supported runner: https://github.com/orgs/community/discussions/19197#discussioncomment-9019461

artoonie avatar Aug 19 '24 19:08 artoonie

Hey, the fixes from the newly merged PR fixing the development build releases should have ran Sunday.

But I am now no longer seeing the development builds at all under releases.

When I checked the GitHub Actions, it looks like the build failed.

nurse-the-code avatar Sep 17 '24 13:09 nurse-the-code

This PR fixes the issue: https://github.com/BrightSpots/rcv/pull/885

artoonie avatar Sep 17 '24 19:09 artoonie

Another automated build kicked off yesterday. I now see three automated builds on the release page. Two I believe were created manually by me clicking the button to test last week. Those are both titled "Draft" and have release dates of last week. One has only the Linux build artifacts from last week, one has Linux build artifacts from the most recent run an hour ago.

Then there is a release titled build-develop: Updates to the per-slice CSV (#872) I'm assuming that is the default now to the take the last PR as the build title? I think the description there is a concatenation of all the commits since the last fortnightly? Or maybe it is just the last commit's message, I'm not sure.

It has MacOSx64 release artifacts only. I think some of the jobs to produce build artifacts are writing over each other.

yezr avatar Oct 01 '24 13:10 yezr