Skip to content

April 2021 Monthly Release #1681

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

Closed
jneira opened this issue Apr 7, 2021 · 24 comments · Fixed by #1702 or #1722
Closed

April 2021 Monthly Release #1681

jneira opened this issue Apr 7, 2021 · 24 comments · Fixed by #1702 or #1722
Labels
CI Continuous integration old_type: meta Planing and organizing other issues type: enhancement New feature or request

Comments

@jneira
Copy link
Member

jneira commented Apr 7, 2021

  • It is release time! @berberman kindly has accepted driving the release this time, many thanks!
  • There are quite pr activity recently and ci transient failures has delayed their handling so we have to wait to reach some stability on that side (i hope it will be soon)
@jneira jneira added CI Continuous integration type: enhancement New feature or request labels Apr 7, 2021
@Ailrun
Copy link
Member

Ailrun commented Apr 7, 2021

Out of curiosity, where did you have such a conversation with @berberman?

@Ailrun Ailrun added the old_type: meta Planing and organizing other issues label Apr 7, 2021
@jneira
Copy link
Member Author

jneira commented Apr 7, 2021

@Ailrun sorry if i was not being transparent or inclusive, we discussed that in irc/fp discord channels. The idea is if more maintainers know about the process the better.
Maybe we should opened the issue as first step? will do next time

@Ailrun
Copy link
Member

Ailrun commented Apr 7, 2021

@jneira Nothing to sorry about, I was just curious about other communication channels, especially active ones.

@jneira jneira linked a pull request Apr 10, 2021 that will close this issue
@wolverian
Copy link

Was this meant to be released as a pre-release?

@berberman
Copy link
Collaborator

Was this meant to be released as a pre-release?

It's the routine of performing release -- once the binaries are ready, we will unmark the pre-release status

@berberman
Copy link
Collaborator

@jneira Ah, CI job failed: https://github.com/haskell/haskell-language-server/runs/2321922916. What should we do next?

@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

Ugh, it did not fail in the two last releases (but it did before that)
As the document suggests the safer (but maybe too careful) is recreate the tag (deleting it from the repo and create it again from the same commit) and the prerelease from zero (deleting the actual one)
Iirc when i tried to simple rerun the build, it throwed errors cause part of the binaries already existed in the release.

NOTE: As stated in the document an alternative could be build the release in an alternative repo and upload manually the missing binary, to no waste the existing ones. But note the binaries used by ghcup (ending in .tar.gz) are not built if one of main ones fails

@berberman
Copy link
Collaborator

It seems that I don't have permission to delete an existing release

@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

Mmm, ok, i've set your user as "maintainer", could you see now a red button close to "Edit release" here: https://github.com/haskell/haskell-language-server/releases/tag/1.1.0?

@berberman
Copy link
Collaborator

yes, now I can

@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

@berberman
Copy link
Collaborator

Damn it, the build failed again

@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

maybe it is a persistent bug, there is no issue with a similar problem?
unfortunately we don't test the build job between releases to check cabal install works correctly (and maybe we should do it)
if you trigger the build in your repo, does it fail? will try myself too

@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

the GitHub actions VM images has been updated recently, maybe it is related

@berberman
Copy link
Collaborator

The runner ubuntu-latest was updated from 18.04 to 20.04 since last release. I will delete the pre-release now and try using 18.04 in my fork

@berberman
Copy link
Collaborator

@jneira Downgrading to 18.04 fixed this: https://github.com/berberman/haskell-language-server/runs/2325193103. So I think we have two choices now:

  1. open a PR to update build.yml, re-tag 1.1.0 on squashed commit of the PR, and finally publish the release
  2. leave build.yml as it is, publish the release, and upload binaries from my fork

2 seems to take less effort, but the commit updating build.yml may affect git rev listed in --probe-tools

@berberman berberman reopened this Apr 12, 2021
@jneira
Copy link
Member Author

jneira commented Apr 12, 2021

I would vote for 1 cause the build will be fixed in master, ready for another release if necessary while we fix it for 20.04.

@berberman
Copy link
Collaborator

berberman commented Apr 13, 2021

Due to the land of #1717, we need prepare a new release of ghcide and re-prepare HLS 1.2 based on #1717, upcoming #1709 & #1715. Now we have a chance to downgrade the ubuntu runner

@berberman
Copy link
Collaborator

It seems that @pepeiborra has already uploaded ghcide-1.2.0.1, but I want to include approved #1709 and #1715 in HLS 1.2 -- both of them made changes in ghcide, so I plan to prepare a ghcide-1.2.0.2 release

@berberman
Copy link
Collaborator

@jneira The build succeeded! So the last step is to upload 8.8.3 and 8.8.4 binaries for Windows?

@jneira
Copy link
Member Author

jneira commented Apr 13, 2021

The build succeeded! So the last step is to upload 8.8.3 and 8.8.4 binaries for Windows?

Nice! Yeah, i'll try to do it this evening GMT (it will take some time in my laptop, with a cold cabal store cache 😟)
Many thanks, you have dealed with this problematic release amazingly

@berberman
Copy link
Collaborator

@jneira I have built 8.8.4 one on my local machine, though it took some efforts... Is it enough? Should we upload 8.8.3 or even 8.8.2 as well?

@jneira
Copy link
Member Author

jneira commented Apr 13, 2021

nice, lately we are uploading only 8.8.4. You can check the last 1.0.0 release, if all artifacts match with the prerelease, it is good to go!

@berberman
Copy link
Collaborator

uploaded and unmarked pre-release now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration old_type: meta Planing and organizing other issues type: enhancement New feature or request
Projects
None yet
4 participants