{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":92529879,"defaultBranch":"main","name":"ghw","ownerLogin":"jaypipes","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2017-05-26T16:39:02.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/265023?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1727264945.0","currentOid":""},"activityList":{"items":[{"before":"c703760f7b059d6bc2b004b487992638d6bc50f6","after":"804ce35e3bf962c8142ade3d1ebbd21aeacf2807","ref":"refs/heads/overhaul-snapshot","pushedAt":"2024-09-25T13:27:17.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"overhaul ghw-snapshot tool\n\nThis is the first step in actually overhauling the ghw library's context\npackage and moving to a more standard `context.Context` usage. In order\nto standardize the context usage, we needed to change the way that the\nghw-snapshot functionality worked to manually construct a\n`pkg/context.Context` and call `pkg/context.Context:Do()` while reading\nthe snapshot tarball.\n\nIn order to do that, I created a new `ghw-snapshot read` command that\naccepts a single argument to the snapshot tarball path to read:\n\n```\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-intel-xeon-L5640.tar.gz\nblock storage (8 disks, 723GB physical storage)\ncpu (2 physical packages, 12 cores, 24 hardware threads)\ngpu (0 graphics cards)\nmemory (66GB physical, 63GB usable)\nnet (0 NICs)\ntopology NUMA (2 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (82 devices)\n\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-amd-ryzen-1600.tar.gz\nblock storage (8 disks, 3TB physical storage)\ncpu (1 physical package, 6 cores, 12 hardware threads)\ngpu (1 graphics card)\nmemory (32GB physical, 32GB usable)\nnet (4 NICs)\ntopology NUMA (0 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (43 devices)\n```\n\nA followup series of patches will make the aforementioned changes to\n`pkg/context` and remove the `pkg/context.Context:Do()` method and put\nthat entirely in the `cmd/ghw-snapshot/command/read.go` file which is\nthe only place we actually use it.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"overhaul ghw-snapshot tool"}},{"before":"e47d613f4253d8aa37e8dac1e0a76243bc4a1787","after":null,"ref":"refs/heads/idiomatic","pushedAt":"2024-09-25T11:49:05.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"ffromani","name":"Francesco Romani","path":"/ffromani","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25349329?s=80&v=4"}},{"before":"29508211e946051c5439ff60d2997cc8a5c82fb6","after":"197a48591077a9528f19f124d2385fe95d046120","ref":"refs/heads/main","pushedAt":"2024-09-25T11:49:04.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"ffromani","name":"Francesco Romani","path":"/ffromani","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25349329?s=80&v=4"},"commit":{"message":"Merge pull request #379 from jaypipes/idiomatic\n\nuse Go idiomatic naming for Architecture consts","shortMessageHtmlLink":"Merge pull request #379 from jaypipes/idiomatic"}},{"before":"da214c29fbe082b576c5b9b1973b7fdb04e0b30b","after":"e47d613f4253d8aa37e8dac1e0a76243bc4a1787","ref":"refs/heads/idiomatic","pushedAt":"2024-09-25T11:46:55.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use Go idiomatic naming for Architecture consts\n\nUpdates the `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA` constants to\n`ArchitectureSMP` and `ArchitectureNUMA`, respectively, to align with\nidiomatic Go naming conventions.\n\nFor backwards-compat, keeps `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA`\nin the aliased variables for a few releases. These will be discarded\nwhen we cut a v1.0 series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use Go idiomatic naming for Architecture consts"}},{"before":"464a1c2a10e1259c42a4cee18f0488c111180d4c","after":null,"ref":"refs/heads/num-total","pushedAt":"2024-09-25T11:41:24.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"26a61afae06e983252578a68cc907f685dfc35cd","after":"29508211e946051c5439ff60d2997cc8a5c82fb6","ref":"refs/heads/main","pushedAt":"2024-09-25T11:41:23.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"Merge pull request #381 from jaypipes/num-total\n\nuse TotalXXX instead of NumXXX","shortMessageHtmlLink":"Merge pull request #381 from jaypipes/num-total"}},{"before":"8a675db9e19a85d39347c9808fdff431b84b2dc3","after":"c703760f7b059d6bc2b004b487992638d6bc50f6","ref":"refs/heads/overhaul-snapshot","pushedAt":"2024-09-24T15:55:18.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"overhaul ghw-snapshot tool\n\nThis is the first step in actually overhauling the ghw library's context\npackage and moving to a more standard `context.Context` usage. In order\nto standardize the context usage, we needed to change the way that the\nghw-snapshot functionality worked to manually construct a\n`pkg/context.Context` and call `pkg/context.Context:Do()` while reading\nthe snapshot tarball.\n\nIn order to do that, I created a new `ghw-snapshot read` command that\naccepts a single argument to the snapshot tarball path to read:\n\n```\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-intel-xeon-L5640.tar.gz\nblock storage (8 disks, 723GB physical storage)\ncpu (2 physical packages, 12 cores, 24 hardware threads)\ngpu (0 graphics cards)\nmemory (66GB physical, 63GB usable)\nnet (0 NICs)\ntopology NUMA (2 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (82 devices)\n\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-amd-ryzen-1600.tar.gz\nblock storage (8 disks, 3TB physical storage)\ncpu (1 physical package, 6 cores, 12 hardware threads)\ngpu (1 graphics card)\nmemory (32GB physical, 32GB usable)\nnet (4 NICs)\ntopology NUMA (0 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (43 devices)\n```\n\nA followup series of patches will make the aforementioned changes to\n`pkg/context` and remove the `pkg/context.Context:Do()` method and put\nthat entirely in the `cmd/ghw-snapshot/command/read.go` file which is\nthe only place we actually use it.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"overhaul ghw-snapshot tool"}},{"before":null,"after":"8a675db9e19a85d39347c9808fdff431b84b2dc3","ref":"refs/heads/overhaul-snapshot","pushedAt":"2024-09-24T13:53:30.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"overhaul ghw-snapshot tool\n\nThis is the first step in actually overhauling the ghw library's context\npackage and moving to a more standard `context.Context` usage. In order\nto standardize the context usage, we needed to change the way that the\nghw-snapshot functionality worked to manually construct a\n`pkg/context.Context` and call `pkg/context.Context:Do()` while reading\nthe snapshot tarball.\n\nIn order to do that, I created a new `ghw-snapshot read` command that\naccepts a single argument to the snapshot tarball path to read:\n\n```\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-intel-xeon-L5640.tar.gz\nblock storage (8 disks, 723GB physical storage)\ncpu (2 physical packages, 12 cores, 24 hardware threads)\ngpu (0 graphics cards)\nmemory (66GB physical, 63GB usable)\nnet (0 NICs)\ntopology NUMA (2 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (82 devices)\n\n➜ ghw git:(overhaul-snapshot) ✗ GHW_DISABLE_WARNINGS=1 go run cmd/ghw-snapshot/main.go read testdata/snapshots/linux-amd64-amd-ryzen-1600.tar.gz\nblock storage (8 disks, 3TB physical storage)\ncpu (1 physical package, 6 cores, 12 hardware threads)\ngpu (1 graphics card)\nmemory (32GB physical, 32GB usable)\nnet (4 NICs)\ntopology NUMA (0 nodes)\nchassis type=unknown vendor=unknown version=unknown\nbios vendor=unknown version=unknown\nbaseboard vendor=unknown version=unknown product=unknown\nproduct family=unknown name=unknown vendor=unknown sku=unknown version=unknown\nPCI (43 devices)\n```\n\nA followup series of patches will make the aforementioned changes to\n`pkg/context` and remove the `pkg/context.Context:Do()` method and put\nthat entirely in the `cmd/ghw-snapshot/command/read.go` file which is\nthe only place we actually use it.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"overhaul ghw-snapshot tool"}},{"before":"aea88a6f6c49e73016054ed5b5ac6bdc30a569ac","after":"464a1c2a10e1259c42a4cee18f0488c111180d4c","ref":"refs/heads/num-total","pushedAt":"2024-09-23T18:58:25.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use TotalXXX instead of NumXXX\n\nAll other fields that contain a total amount of something begin with the\nword `Total` except for the `pkg/cpu.Processor.NumCores` and\n`NumThreads` fields. This commit adds new fields to `pkg/cpu.Processor`\ncalled `TotalCores` and `TotalHardwareThreads` to bring the names of\nthese fields in-line with all other amount/total fields.\n\nIn addition, adds a `TotalHardwareThreads` field to the `pkg/cpu.Info`\nstruct and deprecates the less-explicit `TotalThreads` field.\n\nI'd like to remove the now-deprecated `NumThreads`, `NumCores`, and\n`TotalThreads` fields before we get to a `v.1.0` series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use TotalXXX instead of NumXXX"}},{"before":"1d6b843ea1d056813019977be09d27e0dd5bd6c9","after":"aea88a6f6c49e73016054ed5b5ac6bdc30a569ac","ref":"refs/heads/num-total","pushedAt":"2024-09-23T18:39:30.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use TotalXXX instead of NumXXX\n\nAll other fields that contain a total amount of something begin with the\nword `Total` except for the `pkg/cpu.Processor.NumCores` and\n`NumThreads` fields. This commit adds new fields to `pkg/cpu.Processor`\ncalled `TotalCores` and `TotalHardwareThreads` to bring the names of\nthese fields in-line with all other amount/total fields.\n\nIn addition, adds a `TotalHardwareThreads` field to the `pkg/cpu.Info`\nstruct and deprecates the less-explicit `TotalThreads` field.\n\nI'd like to remove the now-deprecated `NumThreads`, `NumCores`, and\n`TotalThreads` fields before we get to a `v.1.0` series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use TotalXXX instead of NumXXX"}},{"before":null,"after":"1d6b843ea1d056813019977be09d27e0dd5bd6c9","ref":"refs/heads/num-total","pushedAt":"2024-09-23T18:37:00.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use TotalXXX instead of NumXXX\n\nAll other fields that contain a total amount of something begin with the\nword `Total` except for the `pkg/cpu.Processor.NumCores` and\n`NumThreads` fields. This commit adds new fields to `pkg/cpu.Processor`\ncalled `TotalCores` and `TotalHardwareThreads` to bring the names of\nthese fields in-line with all other amount/total fields.\n\nIn addition, adds a `TotalHardwareThreads` field to the `pkg/cpu.Info`\nstruct and deprecates the less-explicit `TotalThreads` field.\n\nI'd like to remove the now-deprecated `NumThreads`, `NumCores`, and\n`TotalThreads` fields before we get to a `v.1.0` series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use TotalXXX instead of NumXXX"}},{"before":"392ebbd1aa67f320e372da0443b5e0a8de8ebcad","after":"da214c29fbe082b576c5b9b1973b7fdb04e0b30b","ref":"refs/heads/idiomatic","pushedAt":"2024-09-23T17:10:39.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use Go idiomatic naming for Architecture consts\n\nUpdates the `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA` constants to\n`ArchitectureSMP` and `ArchitectureNUMA`, respectively, to align with\nidiomatic Go naming conventions.\n\nFor backwards-compat, keeps `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA`\nin the aliased variables for a few releases. These will be discarded\nwhen we cut a v1.0 series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use Go idiomatic naming for Architecture consts"}},{"before":null,"after":"392ebbd1aa67f320e372da0443b5e0a8de8ebcad","ref":"refs/heads/idiomatic","pushedAt":"2024-09-23T17:05:35.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"use Go idiomatic naming for Architecture consts\n\nUpdates the `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA` constants to\n`ArchitectureSMP` and `ArchitectureNUMA`, respectively, to align with\nidiomatic Go naming conventions.\n\nFor backwards-compat, keeps `ARCHITECTURE_SMP` and `ARCHITECTURE_NUMA`\nin the aliased variables for a few releases. These will be discarded\nwhen we cut a v1.0 series.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"use Go idiomatic naming for Architecture consts"}},{"before":"4ba713ab6d5d5158914446a5d12c84aed353043b","after":null,"ref":"refs/heads/linter-fix","pushedAt":"2024-09-21T16:10:14.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"aa56a5375bb502aa46d22fe2e445ffb4da122e31","after":"26a61afae06e983252578a68cc907f685dfc35cd","ref":"refs/heads/main","pushedAt":"2024-09-21T16:10:13.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"Merge pull request #378 from jaypipes/linter-fix\n\nupdate golangci-lint","shortMessageHtmlLink":"Merge pull request #378 from jaypipes/linter-fix"}},{"before":null,"after":"4ba713ab6d5d5158914446a5d12c84aed353043b","ref":"refs/heads/linter-fix","pushedAt":"2024-09-21T16:07:53.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"update golangci-lint\n\nUpdates our linter to a more modern Github Action and underlying\ngolanglint-ci binary.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"update golangci-lint"}},{"before":"f62e6fbcfaad7a64b536180d555d0924b8836cbc","after":null,"ref":"refs/heads/fix-341","pushedAt":"2024-09-21T15:51:07.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"3699cf762e4eac63d21fff37715813e3cb96ff41","after":null,"ref":"refs/heads/issue-345","pushedAt":"2024-09-21T15:51:05.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"1eb70925d800214ff19be43c6db73b0047140fb5","after":null,"ref":"refs/heads/prep-for-v1","pushedAt":"2024-09-21T15:51:03.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"ac4a7e1d69b9a260bfbcf60aa410f15d6637cbc0","after":null,"ref":"refs/heads/cleanup","pushedAt":"2024-09-21T15:44:12.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"67825f054f7a56af451a4e368acebd3d2857133f","after":"aa56a5375bb502aa46d22fe2e445ffb4da122e31","ref":"refs/heads/main","pushedAt":"2024-09-21T15:44:10.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"Merge pull request #377 from jaypipes/cleanup\n\nupdate Step Security hardener action","shortMessageHtmlLink":"Merge pull request #377 from jaypipes/cleanup"}},{"before":"31b06b0ef9a8095623b9fad9b239aaefa3ef1011","after":"ac4a7e1d69b9a260bfbcf60aa410f15d6637cbc0","ref":"refs/heads/cleanup","pushedAt":"2024-09-21T15:40:25.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"update Step Security hardener action\n\nThe step security Github Action step was stuck and not letting the\nworkflow proceed. Step Security advised updating to the latest\nharden-runner version.\n\nIssue step-security/harden-runner#466\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"update Step Security hardener action"}},{"before":null,"after":"31b06b0ef9a8095623b9fad9b239aaefa3ef1011","ref":"refs/heads/cleanup","pushedAt":"2024-09-21T13:10:29.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"disable Step Security hardener action\n\nThe step security Github Action step just continues to get completely\nstuck and won't let the workflow proceed. Disabling it until we can\nfigure out what is going on with it.\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"disable Step Security hardener action"}},{"before":"f8b2519482e1d29826fd82e84a8e4f3791ccd582","after":null,"ref":"refs/heads/fix-gpu-test","pushedAt":"2024-09-20T14:40:05.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"ffromani","name":"Francesco Romani","path":"/ffromani","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25349329?s=80&v=4"}},{"before":"30c83ba904e9847333c12b59d1dc9d2ffc63aaa9","after":"67825f054f7a56af451a4e368acebd3d2857133f","ref":"refs/heads/main","pushedAt":"2024-09-20T14:38:44.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"Merge pull request #376 from jaypipes/fix-gpu-test\n\nsupply override PCIDB_PATH env var in unit tests","shortMessageHtmlLink":"Merge pull request #376 from jaypipes/fix-gpu-test"}},{"before":null,"after":"f8b2519482e1d29826fd82e84a8e4f3791ccd582","ref":"refs/heads/fix-gpu-test","pushedAt":"2024-09-20T14:31:02.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"},"commit":{"message":"supply override PCIDB_PATH env var in unit tests\n\nSome of the unit tests inspect snapshot filesystems and for the GPU\nInfo-collection, the `github.com/jaypipes/pcidb` library is used to get\nPCI information. That library no longer (as of version v0.9) does an\nautomatic fetch of the canonical pci.ids file if it cannot find a\npci.ids file in the local filesystem. It no longer does that network\nfetch because the upstream maintainer of the pci.ids database noticed\ntoo many fetches coming from projects that imported `ghw` which\ntransitively imported `pcidb`.\n\nBecause of this change in behaviour of no longer doing an automatic\nnetwork fetch of the pci.ids database, a unit test in `ghw` that was\nrelying on the pcidb library was no longer working. This patch fixes two\nthings related to the PCI handling in the `pkg/gpu` package. First, it\ncleans up the parsing of the `/sys/class/drm` symlink contents by\nlooking through the path parts and attempting a regex match for a PCI\naddress. Second, it adds an env-var override to the GPU unit tests to\npoint `github.com/jaypipes/pcidb` to a specific pci.ids file that I've\nincluded in the `testdata/` directory.\n\nCloses Issue #375\n\nSigned-off-by: Jay Pipes ","shortMessageHtmlLink":"supply override PCIDB_PATH env var in unit tests"}},{"before":"1baabf819efcf164970c1a548197c636e8780151","after":null,"ref":"refs/heads/context","pushedAt":"2024-09-20T13:08:48.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"d0958f4a3bee5cb7b54fab479c1327df3709610e","after":null,"ref":"refs/heads/revert-iscsi","pushedAt":"2024-09-20T13:08:46.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"6c247d74f7edc85b5eb15c1f74a6e79c2c4147e3","after":null,"ref":"refs/heads/docs","pushedAt":"2024-09-20T13:08:44.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}},{"before":"7c8936e002e80cdabfad8112ed4e33b52c0beb4f","after":null,"ref":"refs/heads/issue-267","pushedAt":"2024-09-20T13:08:42.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jaypipes","name":"Jay Pipes","path":"/jaypipes","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/265023?s=80&v=4"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"startCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yNVQxMzoyNzoxNy4wMDAwMDBazwAAAATAXNst","endCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yMFQxMzowODo0Mi4wMDAwMDBazwAAAAS7-iq2"}},"title":"Activity · jaypipes/ghw"}