Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    René Ribaud
    @uggla
    @bpetit , sounds good to me.
    have you had a chance to look at the other PR about exporter factorization ?
    5 replies
    bpetit
    @bpetit
    @uggla Hi ! I've iterated on your initial PR here: hubblo-org/scaphandre#96
    I've also added a test to check that the prometheus exporter can be scrapped.
    René Ribaud
    @uggla
    ok cool. I want me to review it right ?
    If I can i will have a look this afternoon or this evening.
    :)
    René Ribaud
    @uggla
    Hello @bpetit are you available ?
    14 replies
    bpetit
    @bpetit
    Hi ! @barnumbirr have you been able to test scaphandre on AMD since this thread ? hubblo-org/scaphandre#55
    (with a kernel > 5.11)
    bpetit
    @bpetit

    Just release v0.3.0 : https://github.com/hubblo-org/scaphandre/releases/tag/v0.3.0

    :grinning: Thanks to all the contributors !

    René Ribaud
    @uggla
    @bpetit, I will be on vacation next week so I have some "free" time. If you will have a time slot as well, we could discuss the priority of issues and review them.
    If I will be able to help on something.
    bpetit
    @bpetit
    Hi ! Yeah sure, we can schedule a short meeting on monday or tuesday ?
    René Ribaud
    @uggla
    @bpetit , I apologize I could not make it. I thought I would had some "free" time, but my little daughter disagreed that statement.
    bpetit
    @bpetit
    Do not apologize, family first ! :)
    I may have some time late in the afternoon, or we could schedule a call next week. Tell me what if the best for you :)
    René Ribaud
    @uggla
    Hello @bpetit , I have a slot today 16h --> 17h will it be ok 4 u ?
    7 replies
    Arthur Madec-Prévost
    @arthurmadecprevost
    Hello everyone! Good job for your app. I'm currently testing it and it's such a great thing for a better IT. Do you plan to extend it to other Hypervisors ? Such as VMware, Citrix, Hyper-V ?
    bpetit
    @bpetit

    Hi ! Thanks ! I intend to work on VMWare hypervisors support at some point, but there are several things first in the roadmap. Any help on this would be highly appreciated ! :)

    For Hyper-V first thing would be to add windows support : hubblo-org/scaphandre#74
    Then why not.

    Actually, explanation of supporting only qemu/kvm is not rooted in some deep technical explanation. We just only tested in this context. The data coming from the hypervisor is exposed in the virtual machine through a read only volume, which is not that complicated and should be replicable in vmware (maybe) or other linux based hypervisors.
    Arthur Madec-Prévost
    @arthurmadecprevost
    Thank you for the answer! :D
    Pierre Rust
    @PierreRust
    Hello, I'm playing again with the json exporter
    I have a small question : the consumption in the output is the energy consumed since the previous measurement ?
    in joules ?
    bpetit
    @bpetit
    it is microwatts, it's the power consumption calculated from the energy consumption difference (microjoules) between two measurements and the time step.
    Pierre Rust
    @PierreRust
    ok, thanks for your answer !
    Pierre Rust
    @PierreRust
    I'm currently testing the PR for the json exporter (with timestamp) and there's a few thing I find strange
    the sum of the consumptions for both sockets is not equal to the host consumption
    bpetit
    @bpetit
    which PR ?
    Pierre Rust
    @PierreRust
    yes
    bpetit
    @bpetit
    It's really WIP, it's kind of idle. I wanted to have your thoughts on how it may look like :)
    Could you send me the json result with the bug you mention ?
    so that I look at it
    Pierre Rust
    @PierreRust
    I like it so far :) I currently trying to use it to see if anything important is missing
    here is an example of json :
    {
    "host": {
    "consumption": 25351920,
    "timestamp": 1620112643.986319
    },
    ...
    "sockets": [
    {
    "id": 0,
    "consumption": 15789177,
    "domains": [
    {
    "name": "core",
    "consumption": 1366266
    }
    ]
    },
    {
    "id": 1,
    "consumption": 9785400,
    "domains": [
    {
    "name": "core",
    "consumption": 958565
    }
    ]
    }
    ]
    }
    bpetit
    @bpetit
    I'd like to have your thoughts on this comment too: https://github.com/hubblo-org/scaphandre/issues/75#issuecomment-816623016
    Pierre Rust
    @PierreRust
    that's the way I'm using it right now, looks fine :) I'd rather have a counter instead of a consumption (or both) but that's a detail
    bpetit
    @bpetit
    we could provide both
    regarding the sum, I have to look deeper in the code but I think we dont collect the socket-level consumption exactly at the same time as the host consumption (this is done in the same function but we don't synchronize). So I guess we should provide the timestamp for both kind of measurment
    Pierre Rust
    @PierreRust
    BTW, it seems that the processes' consumption is computed is each exporter, is that not a bit dangerous ? different exporter might expose differents values and a fix in an exporter may not be propagated to others
    bpetit
    @bpetit
    This is not present is that PR as it is quite old, but we have started to create a common interface for exporters at a higher level, to have the same behaviors
    Pierre Rust
    @PierreRust
    or may I just don't understand the code !
    in json.rs :
    consumption: ((*value as f32
                            / (host_time * procfs::ticks_per_second().unwrap() as f32))
                            * host_power as f32),
    20 replies
    bpetit
    @bpetit
    you can see an example in main, in riemann or prometheus. Here it is: https://github.com/hubblo-org/scaphandre/blob/main/src/exporters/riemann.rs#L152
    the functions get_*_metrics
    Pierre Rust
    @PierreRust
    ok, that's a change that is not yet merged in the json exporter PR, is that right ?
    bpetit
    @bpetit
    it is merged in main, but this branch is not up to date yes
    Pierre Rust
    @PierreRust
    Hello, I'm looking right now at the code for the MetricGenerator