profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/TimRots/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

TimRots/ansible-prometheus 0

An Ansible role that installs Prometheus, in the format for Ansible Galaxy.

TimRots/ansible-role-packer-rhel 0

Ansible Role - Packer RHEL/CentOS Configuration for Vagrant VirtualBox

TimRots/ansible-role-prometheus 0

Ansible role to configure prometheus, alertmanager, node_exporter and create_adpater on Centos 7 / Ubuntu 16.04

TimRots/ansible-virtualbox 0

Ansible role for Oracle VM VirtualBox hypervisor

TimRots/appmock 0

Easy CLI mocking

TimRots/containerd 0

An open and reliable container runtime

TimRots/contest 0

Run continuous and on-demand system testing for real and virtual hardware

TimRots/docker-agent 0

Base Docker image for Jenkins Agents

issue commentgosnmp/gosnmp

SNMPv3 trap is not working

用一下1.25.0版本试一试

vgupta-mickey

comment created time in a day

issue openedgosnmp/gosnmp

snmp v2 trap server ceceive v3 trap packet occur panic

when a snmp v2 trap listen received a v3 trap packet,it's occuring follow error:

panic: runtime error: slice bounds out of range [63:61] goroutine 53 [running]: github.com/gosnmp/gosnmp.(*UsmSecurityParameters).unmarshal(0xc0003acc30, 0xc0003acc03, 0xc00065b000, 0xcb, 0x1000, 0x36, 0x10, 0x2499120, 0x448b8d8) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/v3_usm.go:975 +0x12ed github.com/gosnmp/gosnmp.(*GoSNMP).unmarshalV3Header(0xc000496500, 0xc00065b000, 0xcb, 0x1000, 0x1b, 0xc000230380, 0x7, 0x2499120, 0x448b8d8) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/v3.go:414 +0x74f github.com/gosnmp/gosnmp.(*GoSNMP).unmarshalHeader(0xc000496500, 0xc00065b000, 0xcb, 0x1000, 0xc000230380, 0xc000079e60, 0x60529d, 0xc0000100d0) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/marshal.go:965 +0x675 github.com/gosnmp/gosnmp.(*GoSNMP).UnmarshalTrap(0xc000496500, 0xc00065b000, 0xcb, 0x1000, 0x0, 0xc00009f200) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/trap.go:373 +0xa5 github.com/gosnmp/gosnmp.(*TrapListener).listenUDP(0xc0001803c0, 0xc0005809b7, 0x5, 0x0, 0x0) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/trap.go:210 +0x2ae github.com/gosnmp/gosnmp.(*TrapListener).Listen(0xc0001803c0, 0xc0005809b1, 0xb, 0x0, 0x0) #011/root/go/pkg/mod/github.com/gosnmp/gosnmp@v1.32.0/trap.go:348 +0x1dc

suggest:It is recommended to judge the version of snmp first, and ignore packets with inconsistent versions

created time in 2 days

issue commentu-root/u-root

Strategy for keeping Elvish updated to upstream?

@rjoleary, @rminnich: The Elvish project owner was spurred by my proof of concept change to implement an approach they found more palatable; i.e., does not involve custom build tags. You can now generate an elvish binary with no interactive shared history daemon by executing go build ./cmd/nodaemon/elvish using the current master branch. The resulting elvish binary is 16% smaller (7.9M => 6.6M) on macOS and does not have any unwanted artifacts such as unexpected error messages due to the absence of the daemon.

xiaq

comment created time in 3 days

push eventu-root/u-root

Medicine Yeh

commit sha e6b41e7b86dfb817b675313cf900f7e4d2a2e0b8

Print information when netboot failed This can be useful when something goes wrong. The information will be printed in console for later debugging. Signed-off-by: Medicine Yeh <medicinehy@gmail.com>

view details

push time in 3 days

PR merged u-root/u-root

Reviewers
Print information when netboot failed

This can be useful when something goes wrong. The information will be printed in console for later debugging.

+16 -0

2 comments

1 changed file

MedicineYeh

pr closed time in 3 days

issue commentu-root/u-root

u-root .tk domain name

It appears we were afflicted by https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/about-custom-domains-and-github-pages#updating-custom-domains-when-your-github-pages-site-is-disabled

In https://github.com/u-root/u-root/commit/0f27350795cce9bcb98642b0ebdcdea7a5f143c6#diff-00d22ea612a748d8ddd4e5bf6a203591ae83569e7b16f50a11fa6d31a5bedddd, the tk domain was removed from the CNAME file. The DNS record still points to github, but unassigned to a project. I found a project which added a CNAME file to their project to claim our domain. Beside u-root, there are a number of other sites now "Owned by ALI" -- I assume it is some form of unfriendly bot doing this automatically.

I will see if I can resolve this with github support.

mricon

comment created time in 4 days

PR opened u-root/u-root

curl: enable fetch without cache
+88 -12

0 comment

2 changed files

pr created time in 4 days

pull request commentu-root/u-root

Print information when netboot failed

Codecov Report

Merging #2050 (72868cd) into master (ec6777a) will increase coverage by 0.00%. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2050   +/-   ##
=======================================
  Coverage   54.25%   54.25%           
=======================================
  Files         555      555           
  Lines       44656    44656           
=======================================
+ Hits        24226    24228    +2     
+ Misses      19532    19530    -2     
  Partials      898      898           
Impacted Files Coverage Δ
cmds/core/elvish/edit/edit.go 50.83% <0.00%> (-1.25%) :arrow_down:
pkg/strace/tracer.go 57.14% <0.00%> (+2.64%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ec6777a...72868cd. Read the comment docs.

MedicineYeh

comment created time in 5 days

pull request commentu-root/u-root

Print information when netboot failed

The process of build_and_test Expected — Waiting for status to be reported is there for a whole day. How do I trigger it again?

MedicineYeh

comment created time in 6 days

issue openedgosnmp/gosnmp

Breaking change in version 1.32

Version 1.32 introduces a breaking change on PR #329, since it changes the Logger interface. Since the module is marked as stable this introduces some issues on (direct and indirect) consumers of this module because of Go minimal version selection (e.g. one can't use v1.32 of the module if you have a dependency that uses v1.31 's Logger).

One possible way to prevent this in the future is to use apidiff or a similar tool on the CI.

created time in 6 days

issue commentu-root/u-root

Strategy for keeping Elvish updated to upstream?

TODO: eliminating the unused bbolt DB code.

I implemented that as well. I haven't pushed it for review because it shaves only 66K on macOS (64K on Linux x64) from the size of the elvish binary. That's merely a 0.9% decrease in size; which is hard to justify given that it adds 127 lines of code and requires running this command before building elvish to take advantage of the bbolt elimination:

go mod edit -replace go.etcd.io/bbolt ./bbolt_stub

A much more radical refactoring of the code would avoid the need for the go.mod replace directive but that is even harder to justify.

xiaq

comment created time in 7 days

PR opened u-root/u-root

Reviewers
Print information when netboot failed

This can be useful when something goes wrong. The information will be printed in console for later debugging.

+16 -0

0 comment

1 changed file

pr created time in 7 days

pull request commentu-root/u-root

cmds/exp/rush: add changes for tamago

Codecov Report

Merging #2048 (ee98770) into master (78ac944) will decrease coverage by 0.00%. The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2048      +/-   ##
==========================================
- Coverage   54.25%   54.24%   -0.01%     
==========================================
  Files         555      556       +1     
  Lines       44653    44658       +5     
==========================================
  Hits        24226    24226              
- Misses      19529    19534       +5     
  Partials      898      898              
Impacted Files Coverage Δ
cmds/exp/rush/parse.go 0.00% <0.00%> (ø)
cmds/exp/rush/info.go 33.33% <33.33%> (ø)
cmds/exp/rush/rush.go 3.60% <100.00%> (+1.76%) :arrow_up:
cmds/exp/rush/time.go 5.00% <100.00%> (ø)
cmds/core/elvish/edit/edit.go 50.83% <0.00%> (-1.25%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 78ac944...ee98770. Read the comment docs.

rminnich

comment created time in 7 days

issue commentu-root/u-root

Strategy for keeping Elvish updated to upstream?

If https://github.com/elves/elvish/pull/1337 is merged as is you'll be able to build an elvish binary with no daemon (and thus no RPC code or creation of a socket) by running make get ELVISH_MAKE_BIN=./elvish ELVISH_MAKE_TAGS=elv_daemon_stub. This reduces the size of the binary 15% on macOS.

TODO: eliminating the unused bbolt DB code.

xiaq

comment created time in 7 days

pull request commentu-root/u-root

Switch to upstream elvish

If https://github.com/elves/elvish/pull/1337 is merged as is you'll be able to build an elvish binary with no daemon (and thus no RPC code or creation of a socket) by running make get ELVISH_MAKE_BIN=./elvish ELVISH_MAKE_TAGS=elv_daemon_stub. This reduces the size of the binary 15% on macOS.

TODO: eliminating the unused bbolt DB code.

rjoleary

comment created time in 7 days

issue commentu-root/u-root

Automatically regenerate man command from the u-root command

You can see the list of commands and html documentation here: https://pkg.go.dev/github.com/u-root/u-root@master#section-directories (the formatting used to be nicer on godoc.org)

rjoleary

comment created time in 7 days

PR closed u-root/u-root

Readme: Make sure u-root finds the commands

Need to use GO111MODULE=off for fetching, otherwise it won't download the sources into the GOPATH.

+13 -1

1 comment

1 changed file

JohnAZoidberg

pr closed time in 8 days

pull request commentu-root/u-root

Readme: Make sure u-root finds the commands

Hm, actually that doesn't quite work either. It's more difficult than that and depends on the go version (if it has modules or not).

JohnAZoidberg

comment created time in 8 days

PR opened u-root/u-root

Readme: Make sure u-root finds the commands

Need to use GO111MODULE=off for fetching, otherwise it won't download the sources into the GOPATH.

+13 -1

0 comment

1 changed file

pr created time in 8 days

issue commentu-root/u-root

Check formatting of command documentation

Not sure if you imply that also, but I think every command should have at least minimal documentation.

rjoleary

comment created time in 8 days

issue commentu-root/u-root

Automatically regenerate man command from the u-root command

It would be great to also have the manpages also available outside of the shell. For example on a documentation website, like the LinuxBoot book. Perhaps we can have a simple script to turn the extracted manuals to Markdown or first to roff and then HTML/markdown.

I'm trying to figure out how to run the generation script but I can't figure it out: https://github.com/u-root/u-root/pull/2032#issuecomment-861128388

rjoleary

comment created time in 8 days

pull request commentu-root/u-root

Regenerate man command

I can't figure out how you generated that, can you please explain?

Something like go run cmds/core/man/gen/gen.go cmds/core cmds/core/man/data/data.go? But this results in data.go with an empty variable.

> go run cmds/core/man/gen/gen.go cmds/core/free/free.go cmds/core/man/data/data.go
named files must all be in one directory; have cmds/core/man/gen/ and cmds/core/free/

Doesn't work either. Perhaps this is due to Go 1.16? It seems like this is an error by the go command and not the gen.go file.

rjoleary

comment created time in 8 days

delete branch u-root/u-root

delete branch : fix_cmdline

delete time in 8 days

push eventu-root/u-root

Jens Drenhaus

commit sha a7e44e2a1ea35a91574e55341fa1eebb50be6c5e

init: try reading /proc/cmdline after libinit.CreateRootfs() Signed-off-by: Jens Drenhaus <jens.drenhaus@9elements.com>

view details

Jens Drenhaus

commit sha 78ac94480c2b1800531f5e3d00a8826d8ea76276

README: add more information about passing arguments to uinit command Signed-off-by: Jens Drenhaus <jens.drenhaus@9elements.com>

view details

push time in 8 days

PR merged u-root/u-root

Reviewers
Fix cmdline Waiting for Reviewer automerge

Commit b6836a4 introduced a call to the cmdline package trying to open proc/cmdline before libinit.CreateRootfs() which result in an error and an empty procCmdLine inside the cmdline package. Since this variable is initialized only once, any further calls to the cmdline package will have no effect (operate on an empty procCmdLine).

The new part of commit b6836a4

// Backwards compatibility for the transition from uroot.nohwrng to
// UROOT_NOHWRNG=1 on kernel commandline.
if cmdline.ContainsFlag("uroot.nohwrng") {
	os.Setenv("UROOT_NOHWRNG", "1")
	log.Printf("Deprecation warning: use UROOT_NOHWRNG=1 on kernel cmdline instead of uroot.nohwrng")
}

just needs to go anywhere after the rootfs was created. This approach here is just a proposal, feel free to suggest alternatives.

+34 -7

3 comments

3 changed files

jensdrenhaus

pr closed time in 8 days

Pull request review commentu-root/u-root

Fix cmdline

 qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nogr # Go Gopher # ~/> ```+Passing the command line arguments like above is eqivalent to passing the arguments to uinit via a flags file in `/etc/uinit.flags`, see [Extra Files](#extra-files).++Additionally you can pass arguments to uinit via the `uroot.uinitargs` kernel paramteters, for example:++```bash+u-root -uinitcmd="echo Gopher" ./cmds/core/{init,echo,elvish}++cpio -ivt < /tmp/initramfs.linux_amd64.cpio+# ...+# lrwxrwxrwx   0 root     root           12 Dec 31  1969 bin/uinit -> ../bbin/echo+# lrwxrwxrwx   0 root     root            9 Dec 31  1969 init -> bbin/init++qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nographic -append "console=ttyS0 uroot.uinitargs=Go"+# ...+# [    0.848021] Freeing unused kernel memory: 896K+# 2020/05/01 04:04:39 Welcome to u-root!+#                              _+#   _   _      _ __ ___   ___ | |_+#  | | | |____| '__/ _ \ / _ \| __|+#  | |_| |____| | | (_) | (_) | |_+#   \__,_|    |_|  \___/ \___/ \__|+#+# Go Gopher

Probably not, but so everyone knows...

jensdrenhaus

comment created time in 8 days

push eventu-root/u-root

Jens Drenhaus

commit sha 51380ed0f9af7b9382d49450e527ff1bbea00034

README: add more information about passing arguments to uinit command Signed-off-by: Jens Drenhaus <jens.drenhaus@9elements.com>

view details

push time in 8 days

Pull request review commentu-root/u-root

Fix cmdline

 qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nogr # Go Gopher # ~/> ```+Passing the command line arguments like above is eqivalent to passing the arguments to uinit via a flags file in `/etc/uinit.flags`, see [Extra Files](#extra-files).++Additionally you can pass arguments to uinit via the `uroot.uinitargs` kernel paramteters, for example:++```bash+u-root -uinitcmd="echo Gopher" ./cmds/core/{init,echo,elvish}++cpio -ivt < /tmp/initramfs.linux_amd64.cpio+# ...+# lrwxrwxrwx   0 root     root           12 Dec 31  1969 bin/uinit -> ../bbin/echo+# lrwxrwxrwx   0 root     root            9 Dec 31  1969 init -> bbin/init++qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nographic -append "console=ttyS0 uroot.uinitargs=Go"+# ...+# [    0.848021] Freeing unused kernel memory: 896K+# 2020/05/01 04:04:39 Welcome to u-root!+#                              _+#   _   _      _ __ ___   ___ | |_+#  | | | |____| '__/ _ \ / _ \| __|+#  | |_| |____| | | (_) | (_) | |_+#   \__,_|    |_|  \___/ \___/ \__|+#+# Go Gopher

wait what? lol I suppose that was an unintended outcome of this... I didn't think people would combine them

jensdrenhaus

comment created time in 8 days

Pull request review commentu-root/u-root

Fix cmdline

 qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nogr # Go Gopher # ~/> ```+Passing the command line arguments like above is eqivalent to passing the arguments to uinit via a flags file in `/etc/uinit.flags`, see [Extra Files](#extra-files).

s/eqivalent/equivalent

jensdrenhaus

comment created time in 8 days

Pull request review commentu-root/u-root

Fix cmdline

 qemu-system-x86_64 -kernel $KERNEL -initrd /tmp/initramfs.linux_amd64.cpio -nogr # Go Gopher # ~/> ```+Passing the command line arguments like above is eqivalent to passing the arguments to uinit via a flags file in `/etc/uinit.flags`, see [Extra Files](#extra-files).++Additionally you can pass arguments to uinit via the `uroot.uinitargs` kernel paramteters, for example:

s/paramteters/parameters

jensdrenhaus

comment created time in 8 days