Code Monkey home page Code Monkey logo

Comments (7)

dustymabe avatar dustymabe commented on June 29, 2024 1

Folded a fix for this into #426.

Thanks @jlebon

One suggestion was to use the structured journal messages ignition logs now but that gets tricky because we're running in a generator.

What's tricky about querying the journal from a generator?

I just wasn't sure if it would work or not, or cause other problems. Looks like it works 🎉

from fedora-coreos-config.

dustymabe avatar dustymabe commented on June 29, 2024

cc @bgilbert. We should get this fixed before the testing release.

from fedora-coreos-config.

dustymabe avatar dustymabe commented on June 29, 2024

One suggestion was to use the structured journal messages ignition logs now but that gets tricky because we're running in a generator. Obviously one solution may be to hoist this stuff into a different place so we don't have that limitation.

from fedora-coreos-config.

arithx avatar arithx commented on June 29, 2024

Whoops, didn't catch that implication when writing the PR sorry.

One alternative could be to temporarily have Ignition write out a separate file indicating that there was no user configuration. Or we could have a fetch-style stage that could exit depending on whether or not there is a provider / user config defined which we could call in the detection script.

from fedora-coreos-config.

cgwalters avatar cgwalters commented on June 29, 2024

One suggestion was to use the structured journal messages ignition logs now but that gets tricky because we're running in a generator.

What's tricky about querying the journal from a generator?

(The reason this works is because this generator is running in the real root, Ignition runs in the initrd, so it's not going to race or anything)

from fedora-coreos-config.

jlebon avatar jlebon commented on June 29, 2024

Folded a fix for this into #426.

from fedora-coreos-config.

jlebon avatar jlebon commented on June 29, 2024

One thing related to this is that we should fix (or check if it's already fixed by the recent work to close races) the iso-live-login scenario in testiso and turn that on in CI for this repo (and really, all the default testiso scenarios).

from fedora-coreos-config.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.