by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Aug 13 06:31
    dependabot-preview[bot] labeled #166
  • Aug 13 06:31
    dependabot-preview[bot] opened #166
  • Aug 13 06:31

    dependabot-preview[bot] on nuget

    Bump Docker.DotNet from 3.125.2… (compare)

  • Aug 12 17:44

    dependabot-preview[bot] on nuget

    (compare)

  • Aug 12 17:44

    dependabot-preview[bot] on dev

    Bump Microsoft.NET.Test.Sdk fro… (compare)

  • Aug 12 17:44
    dependabot-preview[bot] closed #164
  • Aug 12 17:44
    Aaronontheweb commented #164
  • Aug 12 17:24
    dependabot-preview[bot] synchronize #164
  • Aug 12 17:24

    dependabot-preview[bot] on nuget

    Bump Microsoft.NET.Test.Sdk fro… (compare)

  • Aug 12 17:24
    dependabot-preview[bot] edited #164
  • Aug 12 17:23
    dependabot-preview[bot] edited #164
  • Aug 12 17:23

    dependabot-preview[bot] on nuget

    (compare)

  • Aug 12 17:23

    dependabot-preview[bot] on dev

    Bump System.Data.SqlClient from… (compare)

  • Aug 12 17:23
    dependabot-preview[bot] closed #165
  • Aug 12 17:05
    dependabot-preview[bot] synchronize #165
  • Aug 12 17:05

    dependabot-preview[bot] on nuget

    Bump System.Data.SqlClient from… (compare)

  • Aug 12 17:05
    dependabot-preview[bot] edited #165
  • Aug 12 17:04
    dependabot-preview[bot] edited #165
  • Aug 12 17:04
    dependabot-preview[bot] edited #165
  • Aug 12 17:04
    Aaronontheweb commented #165
Aaron Stannard
@Aaronontheweb
technically, this means a quarantined node is considered "writeable"
total amount of time this code has been in Akka.Remote with this setting: 638 days
public bool HasWriteableEndpointFor(Address address)
        {
            var policy = WritableEndpointWithPolicyFor(address);
            return policy is EndpointManager.Pass;
        }
fixed
Joshua Benjamin
@annymsMthd
Ouch
Aaron Stannard
@Aaronontheweb
yeah, this has got to be at the root of it
there's a few more issues like this
like I said, dumb stuff
Joshua Benjamin
@annymsMthd
It happens.
Aaron Stannard
@Aaronontheweb
I'm going to harden some of the unit tests around this code today too
there's like, obviously bad shit that should not be in here
Joshua Benjamin
@annymsMthd
Nice. We should talk about akka android soon. We are going to be installing our new app in buses soon
Aaron Stannard
@Aaronontheweb
Guess I should talk to our lawyers about liability insurance while we're at it
just kidding :p
but yeah, I would like to see some official support for that
might not be a bad idea to move the ConfigurationManager stuff out of the main repo as part of 1.5
will help with Xamarin support as well as .NET Core
put that stuff into a separate library that's only referenced on full-blown .NET installations
can probably do that part without needing a second nuget package
just an extra binary that only gets added to full .NET profile installs
although, nuts - kind of do need that for the automatic config loading
meh, going to need to just do #IFDEF macros then I guess
we can pretend we're C++ programmers and be all smug about it
Arjen Smits
@Danthar
resistence is futile :P
Vagif Abilov
@object
@Horusiath this is my case too - JSON.NET wouldn't work with my example, this is why I am trying to configure Wire. But don't have that machine now with me, will investigate tomorrow.
Chris Martin
@trbngr
Where did ReplayMessagesSuccess go?
Chris Martin
@trbngr
I guess that was renamed to RecoverySuccess akkadotnet/akka.net@93af339
Aaron Stannard
@Aaronontheweb
@Silv3rcircl3 weird, having this spec fail locally now for some reason: EventFilterTestBase
a couple different implementations of it - never seen that before
ever had any problems with that?
Marc Piechura
@marcpiechura
@Aaronontheweb you mean the remote spec ?
Aaron Stannard
@Aaronontheweb
no
these are testkit specs
SS coming
blob
Marc Piechura
@marcpiechura
Tbh I don't know anything about it, I know there was the PR which added the overload for a custom ActorSystem but haven't run the tests locally for a long time
Aaron Stannard
@Aaronontheweb
I had it fail locally on ./build.cmd all which is bad
since that's build server stuff
Marc Piechura
@marcpiechura
Why do you think I know something about it ? :)
Aaron Stannard
@Aaronontheweb
shows you were one the last people to touch it :p
Marc Piechura
@marcpiechura
o_O
Aaron Stannard
@Aaronontheweb
blob
I think I've found the issue
 protected EventFilterTestBase(string config)
            : base(@"akka.loggers = [""" + typeof(ForwardAllEventsTestEventListener).AssemblyQualifiedName + @"""], " + (string.IsNullOrEmpty(config) ? "" : config))
        {
            //We send a ForwardAllEventsTo containing message to the TestEventListenerToForwarder logger (configured as a logger above).
            //It should respond with an "OK" message when it has received the message.
            var initLoggerMessage = new ForwardAllEventsTestEventListener.ForwardAllEventsTo(TestActor);
            // ReSharper disable once DoNotCallOverridableMethodsInConstructor
            SendRawLogEventMessage(initLoggerMessage);
            ExpectMsg("OK");
            //From now on we know that all messsages will be forwarded to TestActor
        }
assertion in the constructor of the test
Marc Piechura
@marcpiechura
Man I really can't remember that I ever touched this code, will check my PR's from this time
Aaron Stannard
@Aaronontheweb
smells to me like a race condition where something isn't started before this command fires
but not otherwise
works fine when I debug it
Marc Piechura
@marcpiechura
Sorry can't look into it at the moment, can you show me in which PR I changed that? I can't find it