JasperFx/oakton

Jeremy D. Miller

@jeremydmiller

And folks are using Oakton to run .Net Core projects in production, yes. I’ve done that as well

David Smith

@flipdoubt

Excellent. So how do you invoke the describe command on a dotnet core executable compiled for release? For me, app.exe --describe or app.exe --help return without any console output.

Jeremy D. Miller

@jeremydmiller

app.exe describe

app.exe help should give you the help output

You wouldn’t need the double dash if you’re running from the executable instead of dotnet run

David Smith

@flipdoubt

Ah, ok. When I run app.exe help on my dev machine, my app logs the same FileNotFoundException referencing "Microsoft.CodeAnalysis". Now that I can reproduce that locally, I will go through the cleaning exercise.

Jeremy D. Miller

@jeremydmiller

So we still have the Nuget problem. You might see what version of that nuget gets published by dotnet publish

David Smith

@flipdoubt

Same result.

Which package are you referring to with "you might see what version of that nuget gets published by dotnet publish"? There is no Microsoft.CodeAnalysis.dll in the publish output.

Jeremy D. Miller

@jeremydmiller

Yeah, and that’s your problem

So when it says FileNotFoundException, it’s unable to load the assembly referenced. You might try making a direct dependency to that nuget package w/ that version

David Smith

@flipdoubt

Got it.

Thanks.

Jeremy D. Miller

@jeremydmiller

Sorry this is being frustrating. We’ve all been there.

bitbonk

@bitbonk

This is super cool, this should be part of Oakton: https://andrewlock.net/viewing-overriden-configuration-values-in-aspnetcore/

Jeremy D. Miller

@jeremydmiller

@bitbonk It is in 4.1 just by dumping out the built in ConfigurationRoot diagnostics, but not as pretty as what he did. Sounds like a good PR:)

bitbonk

@bitbonk

especially the overwritten values view, this is a live saver

Jeremy D. Miller

@jeremydmiller

Yup. I added it specifically for a project at work for exactly that reason.

bitbonk

@bitbonk

I'll see if I can get a PR ready, just might need to ask if I may steal the code from that post

Jeremy D. Miller

@jeremydmiller

I don’t think he’d mind, and it should be a quick win. should

bitbonk

@bitbonk

@jeremydmiller So here we go: JasperFx/oakton#52

David Smith

@flipdoubt

After changing the standard CreateHostBuilder(args).Build().Run() to CreateHostBuilder(args).RunOaktonCommands(args), some of the our middleware get loggers without log sinks configured. All I have to do is change to code back to fix it, so Build().Run(args) does something RunOaktonCommands(args) does not do. Any clue what that might be and whether there is a work around? It's as if Oakton doesn't call IHostBuilder.Build, but it is hard to tell.

Jeremy D. Miller

@jeremydmiller

@flipdoubt I think it’s more likely that what Oakton is doing when it tries to set the log level dynamically is messing with you.

David Smith

@flipdoubt

@jeremydmiller , I'm "good" with that too, though still confused because it only affects some of our loggers. Do you have any suggestions for a work around? How/where can I help?

Jeremy D. Miller

@jeremydmiller

If there isn’t one already, could you raise a bug in GitHub?

In the Oakton codebase, do a find usages on the NetInput.LogLevelFlag and see where it might be using that too aggressively. I’m inclined to blame Oakton’s special sauce here.

Like it may be clearing logging when it shouldn’t be.

David Smith

@flipdoubt

Looks like the same issue as #51 . I will look into the code too. Thanks.

Jeremy D. Miller

@jeremydmiller

@flipdoubt Sigh, it’s overdue for me to look at it if you don’t find something

David Smith

@flipdoubt

NetCoreInput.cs line 44 would be my guess, but I'm not sure what the intent is behind the code.

Jeremy D. Miller

@jeremydmiller

The original idea was to be able to change the log level up or down based on the input flag

David Smith

@flipdoubt

I haven't stepped into what HostBuilder.ConfigureLogging does, but does that clear all the individual namespaced filters I've configured in Serilog? I will need to write a sample that uses Serilog.

Jeremy D. Miller

@jeremydmiller

I’m guessing so.

David Smith

@flipdoubt

I've spent a decent chunk of time on it and found that code in NetCoreInput never has a logger value and does not do anything out of the ordinary. Something that Oakton does while loading configuration from appsettings.json interferes with how Serilog configures sinks and forces Serilog to use a SilentLogger as described by serilog/serilog#562 and shown below.

I am able to correct this by configuring Serilog with the preserveStaticLogger: true parameter described here.

Essentially, Oakton and Serilog don't play nice together if your Serilog configuration does something fancy like load sinks from appsettings.

Jeremy D. Miller

@jeremydmiller

Or Serilog does something odd to locate the appsettings.json itself. I’ve had no reports of problems with Oakton plus IConfiguration, so that might actually be pointing the finger at Serilog itself.

David Smith

@flipdoubt

Agreed.

Brian Dukes

@bdukes

I'm setting up a new ASP.NET 5 app and when using the RunOaktonCommands (from Oakton 3.1.0 NuGet package), the site starts and then stops immediately.
❯ dotnet run
Building...
Searching 'Lamar.Diagnostics, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null' for commands
info: Microsoft.Hosting.Lifetime[0]
Now listening on: https://localhost:5001
info: Microsoft.Hosting.Lifetime[0]
Now listening on: http://localhost:5000
info: Microsoft.Hosting.Lifetime[0]
Application started. Press Ctrl+C to shut down.
info: Microsoft.Hosting.Lifetime[0]
Hosting environment: Development
info: Microsoft.Hosting.Lifetime[0]
Content root path: C:\inetpub\wwwroot\iss.local\InnovativeStaffSolutions.EmployeeManagement.Web
info: Microsoft.Hosting.Lifetime[0]
Application is shutting down...

Brian Dukes

@bdukes

If I switch the PackageReference to Oakton.AspNetCore 3.0.0 and continue with using Oakton; then it seems to work, but if I use using Oakton.AspNetCore; running the web app works, but running commands gives null references or empty output 🤷🏻‍♂️

Jeremy D. Miller

@jeremydmiller

Oakton.AspNetCore is obsolete, and stuck back at .Net Core 3.1. I'll try it out tomorrow on a local app, but I haven't seen this behavior at all.

Brian Dukes

@bdukes

👍🏻

Brian Dukes

@bdukes

Here's the repo if it helps: https://github.com/bdukes/oakton-aspnet5-issue-repro

I found that it works with Oakton 3.0.0 (which is what I was getting when I referenced Oakton.AspNetCore), but not with 3.0.1, 3.0.2, or 3.1.0

Jeremy D. Miller

@jeremydmiller

@bdukes When you did that, did you get any Nuget warnings about version mismatches?

Brian Dukes

@bdukes

no, only when I tried to use Oakton 2.0.5

Brian Dukes

@bdukes

That error was:

Determining projects to restore...
C:\inetpub\wwwroot\iss.local\InnovativeStaffSolutions.EmployeeManagement.Web\InnovativeStaffSolutions.EmployeeManagement.Web.csproj : error NU1605: Detected package downgrade: Oakton from 3.0.0 to 2.0.5. Reference the package directly from the project to select a different version.
C:\inetpub\wwwroot\iss.local\InnovativeStaffSolutions.EmployeeManagement.Web\InnovativeStaffSolutions.EmployeeManagement.Web.csproj : error NU1605: InnovativeStaffSolutions.EmployeeManagement.Web -> Lamar.Diagnostics 3.0.2 -> Oakton (>= 3.0.0 && < 4.0.0)
C:\inetpub\wwwroot\iss.local\InnovativeStaffSolutions.EmployeeManagement.Web\InnovativeStaffSolutions.EmployeeManagement.Web.csproj : error NU1605: InnovativeStaffSolutions.EmployeeManagement.Web -> Oakton (>= 2.0.5)
Failed to restore C:\inetpub\wwwroot\iss.local\InnovativeStaffSolutions.EmployeeManagement.Web\InnovativeStaffSolutions.EmployeeManagement.Web.csproj (in 792 ms).

Brian Dukes

@bdukes

@jeremydmiller were you able to take a look at that issue and replicate? let me know if there's any other info I can provide. Thanks!

Brian Dukes

@bdukes

Looking at the difference between 3.0.0 and 3.0.1, it looks like it's probably related to the RunCommand switching to OaktonAsyncCommand (or, rather, the changes to the body of the command when that change happened)

Where communities thrive

People

Repo info

Activity