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

Upgrade Ubuntu 20.04 #1522

Merged

Conversation

adenishchenko
Copy link
Contributor

No description provided.

Toktar and others added 30 commits January 10, 2021 23:11
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
[UP-34]: add LEDGERS_FREEZE and GET_FROZEN_LEDGERS transactions
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
…stNodeDelay

Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
Signed-off-by: Ryan Marsh <ryan.marsh44@gmail.com>
@WadeBarnes
Copy link
Member

@ryMarsh44, @Toktar, @adenishchenko, What is the relationship between this PR and #1509? Can #1509 be closed in favor of this PR?

@WadeBarnes
Copy link
Member

@ryMarsh44, @Toktar, @adenishchenko, @udosson, If I understand correctly, based on the Indy Contributors call today, this PR can be taken out of WIP status. Is that correct?

@Toktar
Copy link
Contributor

Toktar commented Mar 30, 2021

@WadeBarnes, #1509 can be closed. This PR contains the previous one.
I don't think that WIP status should be removed because I recommend merge it after adding GitHub Actions pipelines for Ubuntu 20.04.

@askolesov
Copy link
Member

But all tests pass locally

@WadeBarnes
Copy link
Member

@Toktar, @askolesov, This PR contains GHA workflows @udosson developed for the Ubuntu 20.04 upgrade. Does it not?

@Toktar
Copy link
Contributor

Toktar commented Mar 30, 2021

@WadeBarnes Yes, it is. But GHAs for Ubuntu20.04 doesn't work partly, and we are working on them in my fork https://github.com/Toktar/indy-plenum/actions/runs/701558875
We can start to launch the PR there I think.

@WadeBarnes
Copy link
Member

WadeBarnes commented Mar 30, 2021

@Toktar, Great!. I'm getting the same results. Since you're already working on it I'll leave it to you.

There have been some updates to the workflows that need to be integrated in order to fix some of the issues you are seeing and also integrate the CD portion of the flows that @ianco and @udosson added to the version in the main branch; https://github.com/hyperledger/indy-plenum/blob/master/.github/workflows/build.yaml

Namely these changes need to be integrated while the detection of Ubuntu 16.04 vs 20.04 is retained from the version on this branch:

Comment on lines 2 to 10
on:
push:
branches:
- master
- gha-ubuntu-20.04
pull_request:
branches:
- master
- gha-ubuntu-20.04
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Toktar, also, if you add ubuntu-20.04-upgrade to the list of branches, the workflows will be triggered when you commit to this PR.

adenishchenko and others added 4 commits March 31, 2021 01:49
Signed-off-by: Anton Denishchenko <anton.denishchenko@evernym.com>
Signed-off-by: Anton Denishchenko <anton.denishchenko@evernym.com>
Signed-off-by: Anton Denishchenko <anton.denishchenko@evernym.com>
Signed-off-by: anton.denishchenko <anton.denishchenko@dsr-corporation.com>
@WadeBarnes
Copy link
Member

Testing the latest updates over here; https://github.com/WadeBarnes/indy-plenum/actions/runs/726225931

@askolesov
Copy link
Member

Yeah, it was also running in Anton's repo but failing here for some reason

@adenishchenko
Copy link
Contributor Author

@WadeBarnes Yes, I fixed it and tested in my repo https://github.com/adenishchenko/indy-plenum/actions/runs/718983014. All tests pass but in the PR we still have an issue with a secret, I think.
Error message:
perform an interactive login from a non TTY device
I guess, I don't have permissions for fix it.

@WadeBarnes
Copy link
Member

All tests are passing on Ubuntu 20.04. There is still some work to be done on the GHA workflows, but that can be done on follow-up PRs. I'd like to get this PR merged into the ubuntu-20.04 branch so the GHA images can get built and cached, and we can call the majority of the upgrade/migration work complete.

Any objections, @adenishchenko, @Toktar, @askolesov, @udosson?

@askolesov
Copy link
Member

No objections, but how are we going to do the first release on Ubuntu 16 if we merge it now?

@WadeBarnes
Copy link
Member

@WadeBarnes Yes, I fixed it and tested in my repo https://github.com/adenishchenko/indy-plenum/actions/runs/718983014. All tests pass but in the PR we still have an issue with a secret, I think.
Error message:
perform an interactive login from a non TTY device
I guess, I don't have permissions for fix it.

This is expected. PRs don't have access to the repository secrets.

@WadeBarnes
Copy link
Member

No objections, but how are we going to do the first release on Ubuntu 16 if we merge it now?

This is going into the ubuntu-20.04-upgrade branch, the 16.04 release will be done off another branch.

@askolesov
Copy link
Member

Right, thanks. Too many branches...

@WadeBarnes WadeBarnes changed the title WIP: Upgrade Ubuntu 20.04 Upgrade Ubuntu 20.04 Apr 7, 2021
@WadeBarnes WadeBarnes marked this pull request as ready for review April 7, 2021 15:58
@WadeBarnes WadeBarnes merged commit 79aca5d into hyperledger:ubuntu-20.04-upgrade Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants