https://www.puppet.com/community logo
Join Slack
Powered by
# voxpupuli
  • a

    Alex Fisher

    06/10/2022, 2:54 PM
    You mean at least it's json!! 🙂
  • b

    bastelfreak

    06/10/2022, 2:54 PM
    😄
  • b

    bastelfreak

    06/10/2022, 2:54 PM
    yes
  • a

    Alex Fisher

    06/10/2022, 2:54 PM
    most_differences should be somewhere at the bottom after all the hosts.
  • b

    bastelfreak

    06/10/2022, 2:54 PM
    I think it's not there if I diffed a single host, right?
  • b

    bastelfreak

    06/10/2022, 2:54 PM
    I think it's not there if I diffed a single host, right?
  • b

    bastelfreak

    06/10/2022, 2:58 PM
    aha!
  • b

    bastelfreak

    06/10/2022, 2:58 PM
    so this js error
  • b

    bastelfreak

    06/10/2022, 2:58 PM
    only appears with my reports, not the demo one
  • b

    bastelfreak

    06/10/2022, 2:59 PM
    https://github.com/voxpupuli/puppet-catalog-diff-viewer/blob/4144d831dfaa025b40f9e1d1ade2f974aa2aaec9/catalog_viewer.js#L161 here it says
    Uncaught TypeError: diff.most_differences is undefined
  • b

    bastelfreak

    06/10/2022, 2:59 PM
    https://github.com/voxpupuli/puppet-catalog-diff-viewer/blob/4144d831dfaa025b40f9e1d1ade2f974aa2aaec9/catalog_viewer.js#L161 here it says
    Uncaught TypeError: diff.most_differences is undefined
  • b

    bastelfreak

    06/10/2022, 3:00 PM
    ah that's the array you meant
  • b

    bastelfreak

    06/10/2022, 3:00 PM
    so where did that go
  • a

    Alex Fisher

    06/10/2022, 3:01 PM
    Have you tried with an older version of catalog diff? (Before all the recent merges)
  • b

    bastelfreak

    06/10/2022, 3:02 PM
    I had that since the beginning
  • b

    bastelfreak

    06/10/2022, 3:02 PM
    I think so, at least
  • b

    bastelfreak

    06/10/2022, 3:02 PM
    lets say: I saw this error since I implemented the minimal amount of patches required to get it running in this "special" environment
  • a

    Alex Fisher

    06/10/2022, 3:05 PM
    most_differences should exist even if diffing a single node.
  • b

    bastelfreak

    06/10/2022, 3:05 PM
    I have node_differences
  • b

    bastelfreak

    06/10/2022, 3:05 PM
    but that's an int
  • b

    bastelfreak

    06/10/2022, 3:06 PM
    aha!
  • b

    bastelfreak

    06/10/2022, 3:07 PM
    yeah if I add that entry to the json it works
  • b

    bastelfreak

    06/10/2022, 3:07 PM
    so where did it get lost
  • a

    Alex Fisher

    06/10/2022, 3:07 PM
    yeah, I'm looking for culprits.
  • a

    Alex Fisher

    06/10/2022, 3:07 PM
    brb (need to get dinner out of freezer!)
  • a

    Alex Fisher

    06/10/2022, 3:11 PM
    I may have found it. Just double checking before I start pointing fingers! 🙂
  • b

    bastelfreak

    06/10/2022, 3:12 PM
    I see the nodes with most differences on stdout though
  • b

    bastelfreak

    06/10/2022, 3:12 PM
    seems like it's only missing in the report
  • a

    Alex Fisher

    06/10/2022, 3:12 PM
    indeed
  • a

    Alex Fisher

    06/10/2022, 3:12 PM
    @natemccurdy Sorry bud. I think this one's on you! 😉
    sadluke 1
1...697071...648Latest