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

CI issues #153

Closed
knocte opened this issue Apr 11, 2021 · 5 comments
Closed

CI issues #153

knocte opened this issue Apr 11, 2021 · 5 comments

Comments

@knocte
Copy link
Collaborator

knocte commented Apr 11, 2021

I've seen a couple of issues lately with CI:

  • Some lanes fail randomly, e.g.: the one for this recent commit: 4812785
  • In our fork where we test several things, we have noticed that the push to nuget.org can happen even if the other CI lanes didn't pass, which is wrong.
@joemphilips
Copy link
Owner

Let me investigate

@joemphilips
Copy link
Owner

joemphilips commented Apr 12, 2021

The CI failure itself is caused only in "windows && non-bouncycastle" build in Macaroons.
I remember I had similar issue once before.
I can just stop supporting Macaroons in case of non-bouncycastle build (Or maybe I should stop supporting NSec all together).

But before that I want to support .NET 5 and future versions, so please give me some time to research what is the best way to do so.

I'm pretty sure that this issue does not affect you as long as you are using bouncycastle build. So you can ignore it for now. @knocte

@knocte
Copy link
Collaborator Author

knocte commented Apr 12, 2021

this issue does not affect you

Yeah it doesn't matter for us technically, but it's noise when CI fails for reasons outside our control and the workaround is to just re-run CI.

But before that I want to support .NET 5 and future versions, so please give me some time to research what is the best way to do so.

Sure, but DNL is a library, it doesn't have any executables. Therefore it can stay in .NETStandard form and .NET5 programs can use it just fine.

In our fork where we test several things, we have noticed that the push to nuget.org can happen even if the other CI lanes didn't pass, which is wrong.

And how about this?

@knocte
Copy link
Collaborator Author

knocte commented Apr 12, 2021

I can just stop supporting Macaroons in case of non-bouncycastle build

How about just run Macaroon tests in BC build? and don't include it in the nuget package for the nonBC build

joemphilips added a commit that referenced this issue Apr 12, 2021
* Since Macaroons test has annoying Bug which only occurs in "windows * non-bouncycastle build"
* See #153
joemphilips added a commit that referenced this issue Apr 12, 2021
* Since Macaroons test has an annoying bug which only occurs in "windows * non-bouncycastle" build sporadically.
* See #153
joemphilips added a commit that referenced this issue Apr 12, 2021
* Since Macaroons test has an annoying bug which only occurs in "windows * non-bouncycastle" build sporadically.
* See #153
joemphilips added a commit that referenced this issue Apr 12, 2021
* Since Macaroons test has an annoying bug which only occurs in "windows * non-bouncycastle" build sporadically.
* See #153
@joemphilips
Copy link
Owner

Hotfix merged in #155

In the future probably I must consider either

  1. Stop supporting NSec build or
  2. Fixed the test-failure issue on the windows.

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

No branches or pull requests

2 participants