profile
viewpoint
James Hillyerd jhillyerd @google Redmond, WA, US https://twitter.com/jameshillyerd Google Cloud TSE, Go enthusiast

inbucket/inbucket 592

Disposable webmail server (similar to Mailinator) with built in SMTP, POP3, RESTful servers; no DB required.

jhillyerd/enmime 207

MIME mail encoding and decoding package for Go

jhillyerd/plugin-git 168

Git plugin for Oh My Fish (similar to oh-my-zsh git)

inbucket/fauxmailer 7

Generate random emails, sent via SMTP

jhillyerd/perkeep-docker 6

Multi-stage docker build for Perkeep

jhillyerd/go-to-elm-json 3

Convert a Go struct to an Elm record via JSON decoder pipeline.

inbucket/demo.inbucket.org 1

GCP CoreOS config for demo.inbucket.org

jhillyerd/flac2mp3 1

My old FLAC to MP3 script

inbucket/inbucket.org 0

Inbucket Website

jhillyerd/3d-printer-profiles 0

My 3D printer profiles

issue commentjhillyerd/enmime

Why force ASCII for filename when the mime.FormatMediaType will encode the filename properly

It seems in 2018 FormatMediaType did not do what we wanted, see #55

You could try removing it and see how it affects the unit tests.

vertazzar

comment created time in 11 hours

startedbitfield/script

started time in a day

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 3b6a5cbb7bf3594ad8c9016b2179574f9b22e97b

go mod makes golang.fish obsolete

view details

push time in 4 days

PR closed jhillyerd/enmime

WIP: Line wrapping for text/plain content

Implementing text wrapping for text/html is a bit more involved, such that we don't break mid-tag. We should create an issue to track wrapping html content for the future.

closes #63

+299 -18

7 comments

9 changed files

requaos

pr closed time in 4 days

delete branch jhillyerd/enmime

delete branch : develop

delete time in 4 days

delete branch jhillyerd/enmime

delete branch : no-mo-flow

delete time in 4 days

delete branch jhillyerd/enmime

delete branch : bump-go

delete time in 4 days

release jhillyerd/enmime

v0.8.1

released time in 4 days

created tagjhillyerd/enmime

tagv0.8.1

MIME mail encoding and decoding package for Go

created time in 4 days

push eventjhillyerd/enmime

James Hillyerd

commit sha 6eda998e442eb863f7b177f4e7116425504e7c37

Merge branch 'release/0.8.1'

view details

James Hillyerd

commit sha 687195be2805e5ae8ee0656b27f96838afad7621

Merge tag 'v0.8.1' into develop Fixed - Handle incorrectly indented headers (#149, thanks requaos) - Handle trailing separator characters in header (#154, thanks joekamibeppu) Changed - enmime no longer uses git-flow, and will now accept PRs against master

view details

push time in 4 days

push eventjhillyerd/enmime

James Hillyerd

commit sha 11470584bd1939caa6d6a196b9b1633ce961f9dd

Merge tag 'v0.8.0' into develop Added - Inject a `application/octet-stream` as default content type when none is present (#140, thanks requaos) - Add support for content-type params to part & encoding (#148, thanks pzeinlinger) - UTF-7 support (#17) Fixed - Handle missing parameter values in the middle of the media parameter list (#139, thanks requaos) - Fix boundaryReader to respect length instead of capacity (#145, thanks dcormier) - Handle very empty mime parts (#144, thanks dcormier)

view details

Neil

commit sha 2505a646df59dc80689c22c89478327e8177d4fa

Header line incorrectly beginning with a space results in "mime: invalid media parameter" error (#149)

view details

Joe Kamibeppu

commit sha a78482fe1569bdedfdba67cae1332402db503002

Handle trailing separators in header (#154)

view details

James Hillyerd

commit sha f502ae31f30f746a74f4b15a36e3dd2a8df9a172

add shell.nix for nixos based contributors

view details

James Hillyerd

commit sha 852f632ac86bffa203b171082896681c79f4999e

Remove git-flow from contributing guide, closes #151

view details

James Hillyerd

commit sha a76c3697a80f24ba0a37afba91d51f81a53dedab

bump CI golang to 1.14

view details

James Hillyerd

commit sha 4f06961fea31b49b654a4366aafdd94e4aeed475

Update CHANGELOG

view details

James Hillyerd

commit sha 6eda998e442eb863f7b177f4e7116425504e7c37

Merge branch 'release/0.8.1'

view details

push time in 4 days

issue closedjhillyerd/enmime

Migrate off of git-flow

The git-flow strategy of hiding our mainline in a develop branch, and having master reflect the most stable release made sense in a pre-Go-modules world. Now that most projects have adopted Go modules, I don't think we need to protect them from accidentally installing the mainline branch. Other benefits may include:

  • git-flow has been confusing for most new contributors, and they may feel unwelcome when we don't merge their PR to master.
  • The inventor of git-flow noted recently that it doesn't fit many modern workflows.
  • The majority of enhancements to enmime now come from wonderful contributors (big thanks to @dcormier and @requaos), instead of me - my changes should flow through the PR process.
  • With git-flow, the project may appear dead if we have not published a stable release in several months.

I propose we switch to the regular GitHub flow that everyone is used to, and update CONTRIBUTING.md to reflect that.

Let me know if this would break any existing workflows.

closed time in 4 days

jhillyerd

push eventjhillyerd/enmime

James Hillyerd

commit sha 4f06961fea31b49b654a4366aafdd94e4aeed475

Update CHANGELOG

view details

push time in 4 days

push eventjhillyerd/enmime

James Hillyerd

commit sha a76c3697a80f24ba0a37afba91d51f81a53dedab

bump CI golang to 1.14

view details

push time in 4 days

create barnchjhillyerd/enmime

branch : bump-go

created branch time in 4 days

push eventjhillyerd/enmime

James Hillyerd

commit sha f502ae31f30f746a74f4b15a36e3dd2a8df9a172

add shell.nix for nixos based contributors

view details

James Hillyerd

commit sha 852f632ac86bffa203b171082896681c79f4999e

Remove git-flow from contributing guide, closes #151

view details

push time in 4 days

create barnchjhillyerd/enmime

branch : no-mo-flow

created branch time in 4 days

issue commentjhillyerd/enmime

Migrate off of git-flow

@requaos good because I was planning to implement tonight! :)

jhillyerd

comment created time in 4 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 7f1f735e46c4699b8265c8206273991a06e47880

termite: init default config

view details

James Hillyerd

commit sha 8c4c3ff36ccbcd5fd1eac14af2b8a4c3193fd135

termite: set font and colors

view details

James Hillyerd

commit sha 0dfde343dbd038f378358910efc18d91ce162093

replace urxvt with termite

view details

James Hillyerd

commit sha e31505195fe33c538331e9f78e7a07c617acf49b

termite: switch to default colors

view details

push time in 4 days

startedguysoft/OctoPi

started time in 4 days

push eventjhillyerd/Marlin

James Hillyerd

commit sha 862d0b60a28c9cb066541d8ade444b3bb1423af4

tweak manual grid settings

view details

push time in 5 days

create barnchjhillyerd/Marlin

branch : level-margins

created branch time in 5 days

issue closedjhillyerd/enmime

Trailing comma for Content-Type value causes an error

Hi James, thanks for providing and maintaining this great tool! I ran into this while using it and wanted to see what you thought.

What I did:

Run enmime.ReadEnvelope() on a raw email containing the header Content-Type: text/plain;. Note the trailing ; at the end, with nothing following it.

What I expected:

The raw email to be parsed successfully.

What I got:

An error saying, mime: invalid media parameter

Release or branch I am using:

Release v0.8.0

If this is indeed an issue, I believe I have a small fix ready. I could open a PR against the develop branch.

Thanks!

closed time in 5 days

joekamibeppu

pull request commentjhillyerd/enmime

Handle trailing separators in header

Thanks!

joekamibeppu

comment created time in 5 days

push eventjhillyerd/enmime

Joe Kamibeppu

commit sha a78482fe1569bdedfdba67cae1332402db503002

Handle trailing separators in header (#154)

view details

push time in 5 days

PR merged jhillyerd/enmime

Handle trailing separators in header

Hello. This is a fix for https://github.com/jhillyerd/enmime/issues/153.

If my understanding is correct, here is what was happening. When the Content-Type header value was something like text/plain;, it would turn into text/plain;=not-a-param-value, then =not-a-param-value parsed on its own would cause a mime: invalid media parameter error in turn.

Thanks!

+10 -0

1 comment

2 changed files

joekamibeppu

pr closed time in 5 days

create barnchjhillyerd/dotfiles

branch : termite

created branch time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha fc1084e0a3cd90f76edeb9b5bee8534651799764

add cmcd abbr

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 33cf0c7e7764a44765fbb21af0aaa2753fb3c174

polybar: add default config

view details

James Hillyerd

commit sha 591eea46a6c2b1cd6d2fd5ca2bd8498cc3975171

polybar: begin customizing

view details

James Hillyerd

commit sha 025745d34038e1d1b5d3109d594ce016a4a60ef3

add minimal sxhkd config

view details

James Hillyerd

commit sha b8ba5dfcb0b58dab397af1005a074c916e119e2f

add basic bspwm configuration

view details

James Hillyerd

commit sha f32dd90ef41572209a298e79f3b27af981d0edb4

bspwm & polybar config improvements

view details

James Hillyerd

commit sha 5d7ce26acd62e4475ea8a6589682bef0efd02835

hotkey improvements

view details

James Hillyerd

commit sha 21b77d832a3499a39a5d9127014874276f71e0a2

bspwm: add window resize mode

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 21b77d832a3499a39a5d9127014874276f71e0a2

bspwm: add window resize mode

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 5d7ce26acd62e4475ea8a6589682bef0efd02835

hotkey improvements

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 4c7dc34edee64856254e05694fc8b4d8a2e7a60b

vim -d for merge tool

view details

James Hillyerd

commit sha f4ace1583823b1a95437d9c9c568c491c2d0bc36

init: empty weechat config

view details

James Hillyerd

commit sha 96c9bc079ce9e7f6aeb60ffdcf907307e55d1747

weechat: /save config w/ template

view details

James Hillyerd

commit sha cc88f48430221a2e5fc54e015140c4ec866838bf

weechat: add smart filter

view details

James Hillyerd

commit sha d80d04d1159fa08efb3e59a6b42fb42c3442f1ed

add zen book to keys

view details

James Hillyerd

commit sha 33cf0c7e7764a44765fbb21af0aaa2753fb3c174

polybar: add default config

view details

James Hillyerd

commit sha 591eea46a6c2b1cd6d2fd5ca2bd8498cc3975171

polybar: begin customizing

view details

James Hillyerd

commit sha 025745d34038e1d1b5d3109d594ce016a4a60ef3

add minimal sxhkd config

view details

James Hillyerd

commit sha b8ba5dfcb0b58dab397af1005a074c916e119e2f

add basic bspwm configuration

view details

James Hillyerd

commit sha f32dd90ef41572209a298e79f3b27af981d0edb4

bspwm & polybar config improvements

view details

James Hillyerd

commit sha 6f4523e86686c6b2b255414254f24535e39d3567

hotkey improvements

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha d80d04d1159fa08efb3e59a6b42fb42c3442f1ed

add zen book to keys

view details

push time in 6 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 4c7dc34edee64856254e05694fc8b4d8a2e7a60b

vim -d for merge tool

view details

James Hillyerd

commit sha f4ace1583823b1a95437d9c9c568c491c2d0bc36

init: empty weechat config

view details

James Hillyerd

commit sha 96c9bc079ce9e7f6aeb60ffdcf907307e55d1747

weechat: /save config w/ template

view details

James Hillyerd

commit sha cc88f48430221a2e5fc54e015140c4ec866838bf

weechat: add smart filter

view details

push time in 8 days

create barnchjhillyerd/dotfiles

branch : weechat

created branch time in 8 days

issue commentjhillyerd/enmime

Trailing comma for Content-Type value causes an error

Thanks for reporting Joe! We'd definitely appreciate a fix, especially if it includes a unit test so we can prevent a regression in the future.

joekamibeppu

comment created time in 8 days

issue commentnix-community/nixops-libvirtd

Allow running libvirtd deployments on remote hosts

This may already be possible, see:

  • https://github.com/NixOS/nixops/pull/824
  • https://github.com/nix-community/nixops-libvirtd/blob/master/nix/libvirtd.nix#L34

I can try testing and submit some docs if it works.

mbrgm

comment created time in 9 days

startedvlktomas/nix-examples

started time in 9 days

pull request commentnix-community/nixops-libvirtd

Poetry2nixify

@grahamc give these steps a try: https://gist.github.com/jhillyerd/38dd7fe1492b713c64686055c910dfa7

I forgot I had written https://github.com/nix-community/nixops-libvirtd/pull/15/files - but after experimenting last night I've determined the main thing missing is the storage pool for root, not a particular user, so it needs to be updated. So I've flipped that to draft for now.

grahamc

comment created time in 9 days

starteddohliam/dropin-minimal-css

started time in 11 days

create barnchjhillyerd/dotfiles

branch : bspwm

created branch time in 12 days

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 5fbf50cdfebe59cd4ff1825209f0fea784d5d51b

auth keys: add ryzen nix

view details

James Hillyerd

commit sha 884fe4334cf6bdfd8057c9d70aeb41f592aad7f1

only accelerating Chrome's scrolling

view details

push time in 13 days

PublicEvent

push eventjhillyerd/dotvim

James Hillyerd

commit sha e3f5a18607b53f3757e7b5fb1138a0ed61c99dd0

python indents to 4 chars

view details

push time in 14 days

push eventjhillyerd/dotvim

James Hillyerd

commit sha e0f61bab386ae94cf5f48dd40033d9c5325010b6

coc for javascript/ts

view details

push time in 15 days

issue commenttpmullan/OctoPrint-DetailedProgress

Web settings are not persistent when configuration exists in file. (only sending ETA)

Oh. I think that is likely what is confusing everybody then. Labeling the <select> box "Active messages" implies that all of them are active. I'd recommend labeling it "Available messages" and then add some text stating that the selected ones are the active messages.

Perhaps even a warning if the user tries to save with no messages selected, as that's what I've been doing.

Mac2712

comment created time in 15 days

issue commenttpmullan/OctoPrint-DetailedProgress

Web settings are not persistent when configuration exists in file. (only sending ETA)

I just tried deleting ETL via the web interface, and it updated to:

   detailedprogress:
        all_messages:
        - '{filename}'
        - ETA {ETA}
        messages:
        - ETL {printTimeLeft}
        time_to_change: '5'

Prior to https://github.com/tpmullan/OctoPrint-DetailedProgress/issues/8#issuecomment-626182399 - I had deleted the accuracy line via web, so for some reason it overwrites messages: with the deleted entry. The content of all_messages is correct (in terms of what I want the be displayed), but is not used.

I will try removing the detailedprogress section from the config entirely.

Mac2712

comment created time in 15 days

issue commentNixOS/nixops

system.stateVersion differs from default value when not setting explicitely

We should also document if/when to include system.stateVersion in the Nixops docs, as it's not part of the examples and is non-obvious.

datafoo

comment created time in 17 days

push eventjhillyerd/homesite

James Hillyerd

commit sha ac0a33d7d2a8e3a71aef8148436a907f9ddb1662

tsconfig: remove commented out lines

view details

push time in 19 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 187de5a94a76967a1d05b5c22f1545f474a64a33

main: use promised fetch()

view details

James Hillyerd

commit sha 5a510ecf3d0fac072d0b576ccd5370869dc4b468

main: switch to async

view details

push time in 19 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 0ae02cb48652a639bcecc6546d889678d3440ff4

remove npm lockfile

view details

James Hillyerd

commit sha af19cb5d6d62e21fc4a1e2b281a1412f41247db3

Add typescript

view details

James Hillyerd

commit sha 6bfd6ddb3a292d5726d7130b9b106ffbea6633e8

services: convert to typescript

view details

James Hillyerd

commit sha d2047a8890ab5f20685d7692743733cc64c35c45

main: convert to typescript

view details

push time in 19 days

push eventjhillyerd/dotvim

James Hillyerd

commit sha 769c86e98c0b880dea3f310585a058ea87aa2c24

jsonc support

view details

push time in 19 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 28f8e977b8514632304fe2347a06ff18045cdbc4

Add yarn + lock

view details

push time in 20 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 2f209718e7c51750cc32cbac00cabf5ea92a3947

npm: add install script

view details

push time in 20 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 373891da91bfd73275e25697477dfa31b8b39787

style: css grid on desktop

view details

push time in 20 days

create barnchjhillyerd/Marlin

branch : remtime

created branch time in 20 days

push eventjhillyerd/Marlin

Mathias Gartner

commit sha e4d8336175ee09b28d23220f446d07451ca3de43

Fix XYZ types multiplication/division (#17826)

view details

push time in 20 days

delete branch jhillyerd/Marlin

delete branch : custom

delete time in 20 days

create barnchjhillyerd/Marlin

branch : master

created branch time in 20 days

push eventjhillyerd/Marlin

Mathias Gartner

commit sha e4d8336175ee09b28d23220f446d07451ca3de43

Fix XYZ types multiplication/division (#17826)

view details

James Hillyerd

commit sha 81c23b46fcaa07b354e6d8f0a4c23c5c1b1ac966

SKR mini E3 1.2 Marlin defaults

view details

James Hillyerd

commit sha 358d3d41f0808580a35313ef538ca4a855ddd7b7

Align with BTT provided config Note: Advanced Nozzle Park is required for other features, so leaving it enabled even though BTT had it turned off.

view details

push time in 20 days

issue commenttpmullan/OctoPrint-DetailedProgress

only sending ETA

I have a similar problem, mine is configured (according to web UI) to send:

{filename}
ETL {printTimeLeft}
ETA {ETA]

but all that shows on my printers screen is Good accuracy

This is what I see in config.yaml:

    detailedprogress:
        all_messages:
        - '{filename}'
        - ETL {printTimeLeft}
        - ETA {ETA}
        messages:
        - '{accuracy} accuracy'
        time_to_change: '5'

So probably related to #7 as well.

Mac2712

comment created time in 21 days

push eventjhillyerd/homesite

James Hillyerd

commit sha 7af300e05b5fa236342d9323d2813513eeececb9

services: add unknown icon

view details

push time in 21 days

issue commentNixOS/nix

How to manually replicate/reproduce/obtain the sha256 hash specified in Nix with fetchgit or fetchFromGitHub?

This should be added to nixpkgs documentation near https://nixos.org/nixpkgs/manual/#chap-pkgs-fetchers

ghost

comment created time in 21 days

PublicEvent

PR closed NixOS/nixops

known_hosts: Don't update a key to an empty IP address

Remove keys that have an empty IP address instead of trying to re-add them during the update phase.

Should resolve error seen in https://github.com/nix-community/nixops-vbox/pull/3#issuecomment-620104958

+2 -1

3 comments

1 changed file

jhillyerd

pr closed time in 24 days

pull request commentNixOS/nixops

known_hosts: Don't update a key to an empty IP address

Good to close. 👍

jhillyerd

comment created time in 24 days

Pull request review commentNixOS/nixops

known_hosts: Don't update a key to an empty IP address

 def update(prev_address: str, new_address: str, public_host_key: str) -> None:     # FIXME: this rewrites known_hosts twice.     if prev_address != new_address:         remove(prev_address, public_host_key)-    add(new_address, public_host_key)+    if new_address is not None:

Possibly. I don't know a good way to test changes do this code, so I was not comfortable going beyond a break-fix.

My guess is that none of these were meant to be called with empty IP addresses, and that the Virtualbox plugin is broken somewhere.

jhillyerd

comment created time in a month

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 7b28e7876e21c19db0de1d839185383fa39bc9c5

add auth key

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha f1b1bd227b2720052e3955df6aea1fa3e9dc450b

coc: add more mappings, disable ultisnips for now

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha 0140691506c75859080bf1a034a400c58598c76a

Add nix support

view details

James Hillyerd

commit sha 8c50c980e2fe0d95404c20e73849c4b289e145b9

Switch to andys8/vim-elm-syntax

view details

James Hillyerd

commit sha cb0061ecf4a30d841519a4d0a7c6fb3606131fd9

Update ale plugin

view details

James Hillyerd

commit sha e0c51fb08857f2a66ede3e4c330b6ef64b5277a9

initial coc config

view details

James Hillyerd

commit sha 98bddc872dca29bbbddb328a17afff1d475d038d

coc: navigate diags with cursor keys

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha e0c51fb08857f2a66ede3e4c330b6ef64b5277a9

initial coc config

view details

James Hillyerd

commit sha 98bddc872dca29bbbddb328a17afff1d475d038d

coc: navigate diags with cursor keys

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha cb0061ecf4a30d841519a4d0a7c6fb3606131fd9

Update ale plugin

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha 8c50c980e2fe0d95404c20e73849c4b289e145b9

Switch to andys8/vim-elm-syntax

view details

push time in a month

push eventjhillyerd/dotvim

James Hillyerd

commit sha 0140691506c75859080bf1a034a400c58598c76a

Add nix support

view details

push time in a month

pull request commentNixOS/nixops

known_hosts: Don't update a key to an empty IP address

I tested this on nixops-vbox with tests/functional/single_machine_vbox_base.nix as well as a dual machine network of my own with create, deploy, stop (and no stop), destroy, and delete. No issues encountered.

jhillyerd

comment created time in a month

PR opened NixOS/nixops

known_hosts: Don't update a key to an empty IP address

Remove keys that have an empty IP address instead of trying to re-add them during the update phase.

Should resolve error seen in https://github.com/nix-community/nixops-vbox/pull/3#issuecomment-620104958

+2 -1

0 comment

1 changed file

pr created time in a month

push eventjhillyerd/nixops

James Hillyerd

commit sha a1206b718c6375fd9e4eb2bbe470c7d907623b2e

known_hosts: Don't update a key to an empty IP address Remove keys that have an empty IP address instead of trying to re-add them during the update phase. Should resolve error seen in https://github.com/nix-community/nixops-vbox/pull/3#issuecomment-620104958

view details

push time in a month

push eventjhillyerd/nixops

James Hillyerd

commit sha ceae66129e34affc858b58052a694bb3e4178dd1

known_hosts: Don't update a key to ian empty IP address Remove keys that have an empty IP address instead of trying to re-add them during the update phase. Should resolve error seen in https://github.com/nix-community/nixops-vbox/pull/3#issuecomment-620104958

view details

push time in a month

create barnchjhillyerd/nixops

branch : keyupdatefix

created branch time in a month

fork jhillyerd/nixops

NixOps, the NixOS-based cloud deployment tool

https://nixos.org/nixops

fork in a month

pull request commentnix-community/nixops-vbox

Poetry2nixify

Updating nixops to master seems to have broken deploy after a fresh create, I cannot deploy my own or the test network.

machine> creating VirtualBox VM...
machine> Virtual machine 'nixops-056f2440-89db-11ea-9f41-000c297b45f0-machine' is created and registered.
machine> UUID: b45e098a-97e6-4ee5-8b0e-626fa1ea97ca
machine> Settings file: '/home/james/VirtualBox VMs/nixops-056f2440-89db-11ea-9f41-000c297b45f0-machine/nixops-056f2440-89db-11ea-9f41-000c297b45f0-machine.vbox'
Traceback (most recent call last):
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/bin/nixops", line 11, in <module>
    load_entry_point('nixops', 'console_scripts', 'nixops')()
  File "/home/james/devel/nixops/nixops/__main__.py", line 630, in main
    args.op(args)
  File "/home/james/devel/nixops/nixops/script_defs.py", line 605, in op_deploy
    max_concurrent_activate=args.max_concurrent_activate,
  File "/home/james/devel/nixops/nixops/deployment.py", line 1379, in deploy
    self.run_with_notify("deploy", lambda: self._deploy(**kwargs))
  File "/home/james/devel/nixops/nixops/deployment.py", line 1368, in run_with_notify
    f()
  File "/home/james/devel/nixops/nixops/deployment.py", line 1379, in <lambda>
    self.run_with_notify("deploy", lambda: self._deploy(**kwargs))
  File "/home/james/devel/nixops/nixops/deployment.py", line 1286, in _deploy
    worker_fun=worker,
  File "/home/james/devel/nixops/nixops/parallel.py", line 108, in run_tasks
    raise list(exceptions.values())[0]
  File "/home/james/devel/nixops/nixops/parallel.py", line 72, in thread_fun
    work_result = (worker_fun(t), None, t.name)
  File "/home/james/devel/nixops/nixops/deployment.py", line 1245, in worker
    allow_recreate=allow_recreate,
  File "/home/james/devel/nixops-vbox/nixops_vbox/backends/virtualbox.py", line 303, in create
    for sf_name, sf_def in defn.config["virtualbox"]["sharedFolders"].items():
TypeError: 'MachineOptions' object is not subscriptable
grahamc

comment created time in a month

pull request commentnix-community/nixops-vbox

Poetry2nixify

I updated nixops to master via local poetry link, and this still fails with both the test VM and my own VMs.

It may be related to https://github.com/NixOS/nixops/issues/1010 and https://github.com/NixOS/nixops/issues/1279 - it appears to be following the same code path.

Setting a breakpoint, we can see it's the IP that is None, but we are trying to add it:

  File "/home/james/devel/nixops-vbox/nixops_vbox/backends/virtualbox.py", line 595, in destroy
    nixops.known_hosts.update(self.private_ipv4, None, self.public_host_key)
  File "/home/james/devel/nixops/nixops/known_hosts.py", line 69, in update
    add(new_address, public_host_key)
  File "/home/james/devel/nixops/nixops/known_hosts.py", line 62, in add
    _rewrite(ip_address, True, public_host_key)
  File "/home/james/devel/nixops/nixops/known_hosts.py", line 46, in _rewrite
    new.append(ip_address + " " + public_host_key)
TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'
(nixops-vbox-vAlhE9FH-py3.7) 

[james@nixvm:~/devel/nixops-vbox]$ nixops destroy -d test
warning: are you sure you want to destroy VirtualBox VM ‘machine’? (y/N) y
machine> destroying VirtualBox VM...
> /home/james/devel/nixops/nixops/known_hosts.py(47)_rewrite()
-> new.append(ip_address + " " + public_host_key)
(Pdb) args
ip_address = None
add_ip = True
public_host_key = 'ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIPiyDEA2E24ESIlFzMWdKfO7sJujjqwvKSsnzLRL10ul NixOps auto-generated key'
(Pdb) 
grahamc

comment created time in a month

issue commentjhillyerd/enmime

Error decoding attachment file names in utf-8

Please submit a pull request against the develop branch.

int01

comment created time in a month

startedfd0/grobi

started time in a month

push eventjhillyerd/dotfiles

James Hillyerd

commit sha 72e47e49f51ab6d0c4939ad83a2d5b1386411011

chezmoi: more abbrevs

view details

push time in a month

push eventjhillyerd/dotfiles

James Hillyerd

commit sha ce65462f579150187b69480f90a276b616bd5372

git: configure pull to ff

view details

push time in a month

startedmattn/go-isatty

started time in a month

pull request commentnix-community/nixops-vbox

Poetry2nixify

Tested. Successfully deployed a VM.

Destroy failed:


nixops destroy
warning: are you sure you want to destroy VirtualBox VM ‘webserver’? (y/N) y
webserver> destroying VirtualBox VM...
webserver> 0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
Traceback (most recent call last):
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/bin/nixops", line 11, in <module>
    load_entry_point('nixops', 'console_scripts', 'nixops')()
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/__main__.py", line 626, in main
    args.op(args)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/script_defs.py", line 625, in op_destroy
    include=args.include or [], exclude=args.exclude or [], wipe=args.wipe
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/deployment.py", line 1516, in destroy_resources
    "destroy", lambda: self._destroy_resources(include, exclude, wipe)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/deployment.py", line 1378, in run_with_notify
    f()
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/deployment.py", line 1516, in <lambda>
    "destroy", lambda: self._destroy_resources(include, exclude, wipe)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/deployment.py", line 1506, in _destroy_resources
    nr_workers=-1, tasks=list(self.resources.values()), worker_fun=worker
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/parallel.py", line 108, in run_tasks
    raise list(exceptions.values())[0]
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/parallel.py", line 72, in thread_fun
    work_result = (worker_fun(t), None, t.name)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/deployment.py", line 1497, in worker
    if m.destroy(wipe=wipe):
  File "/home/james/devel/nixops-vbox/nixops_vbox/backends/virtualbox.py", line 595, in destroy
    nixops.known_hosts.update(self.private_ipv4, None, self.public_host_key)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/known_hosts.py", line 69, in update
    add(new_address, public_host_key)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/known_hosts.py", line 62, in add
    _rewrite(ip_address, True, public_host_key)
  File "/home/james/.cache/pypoetry/virtualenvs/nixops-vbox-vAlhE9FH-py3.7/src/nixops/nixops/known_hosts.py", line 46, in _rewrite
    new.append(ip_address + " " + public_host_key)
TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'

nix-info system: "x86_64-linux", multi-user?: yes, version: nix-env (Nix) 2.3.4, channels(root): "nixos-20.03.1589.9137f05564e", nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

grahamc

comment created time in a month

push eventjhillyerd/3d-printer-profiles

James Hillyerd

commit sha b4860763d0bb696825d0cf04e54062260d284cce

Initial import of PrusaSlicer profiles

view details

James Hillyerd

commit sha 592c4b5ba7daa045dd5a0ae9a726b982289745b0

Add link to snapshot.ini

view details

James Hillyerd

commit sha 5b903bc1f0862a449f1528aa4ebae326ea788638

Note whambam

view details

push time in a month

push eventjhillyerd/3d-printer-profiles

James Hillyerd

commit sha 60c6c255b3c1463e6d0a3769cd7a662ffaee3fc0

Note whambam

view details

push time in a month

push eventjhillyerd/3d-printer-profiles

James Hillyerd

commit sha 0f3f1a09d9e1cdb974696d95077761ca85512d46

Add link to snapshot.ini

view details

push time in a month

push eventjhillyerd/3d-printer-profiles

James Hillyerd

commit sha 591dfac810131795cb9cbd4816293b2f5175ebfe

Initial import of PrusaSlicer profiles

view details

push time in a month

create barnchjhillyerd/3d-printer-profiles

branch : master

created branch time in a month

created repositoryjhillyerd/3d-printer-profiles

My 3D printer profiles

created time in a month

startedrobcowart/raspberry_pi_stats

started time in a month

issue commentinbucket/inbucket

Configurable URL base path

Note to self: AppConfig is stored in the Elm Session, it would make sense to have Session generate URLs for API calls. Would also need Route to take some sort of base URI parameter.

atmaniak

comment created time in a month

startedinbucket/inbucket

started time in a month

issue commentinbucket/inbucket

Configurable URL base path

When I moved this to the backlog, I didn't realize it would take this long to finish the 3.0 beta... Off the top of my head, I'm not exactly sure how much work this would be. But I think we can make it part of 3.0 instead of waiting for a follow-up release.

The way we bootstrap config from Go to Elm is via this JSON pair, stuffing it into a cookie (hack!), that way no API calls (and URI paths) are involved during UI startup.

  • https://github.com/inbucket/inbucket/blob/master/pkg/server/web/app_json.go
  • https://github.com/inbucket/inbucket/blob/master/ui/src/Data/AppConfig.elm

I'm not yet sure how we'd need to modify the Elm code to make use of this. The nature of functional programming often requires things liked this to be plumbed through, as there is no easy way to get to a global configuration context.

atmaniak

comment created time in a month

PR opened nix-community/nixops-libvirtd

Add Quickstart to README

Provide basic steps to get up and running with libvirtd to help new users get started.

+50 -0

0 comment

1 changed file

pr created time in a month

push eventjhillyerd/nixops-libvirtd

James Hillyerd

commit sha cd5e18b068c1bc74c565c95941929a2bd9f9780b

README: Add 'Prepare libvirtd' section based on nixops 1.7 manual

view details

James Hillyerd

commit sha a444c10543595b8471a8eaa876f2ae0a08f87b75

README: Add 'Deploy the example machine' section

view details

push time in a month

more