Skip to content

zfs: Don't fail parsing of zpool stats if pool health is UNAVAIL on FreeBSD#3149

Merged
danielnelson merged 2 commits intoinfluxdata:masterfrom
rickard-von-essen:zfs-freebsd-unavail
Aug 25, 2017
Merged

zfs: Don't fail parsing of zpool stats if pool health is UNAVAIL on FreeBSD#3149
danielnelson merged 2 commits intoinfluxdata:masterfrom
rickard-von-essen:zfs-freebsd-unavail

Conversation

@rickard-von-essen
Copy link
Contributor

zpool list -Hp will report:

temp2   -       -       -       -       -       -       -       UNAVAIL -
zroot   110595407872    53253271552     57342136320     -       16% 48      1.00x   ONLINE  -

Before this parsing will fail trying to convert - to Int.

Signed-off-by: Rickard von Essen rickard.von.essen@gmail.com

…reeBSD

`zpool list -Hp` will report:

```
temp2   -       -       -       -       -       -       -       UNAVAIL -
zroot   110595407872    53253271552     57342136320     -       16% 48      1.00x   ONLINE  -
```

Before this parsing will fail trying to convert - to Int.

Signed-off-by: Rickard von Essen <rickard.von.essen@gmail.com>
@danielnelson danielnelson added this to the 1.4.0 milestone Aug 22, 2017
@danielnelson
Copy link
Contributor

Can you add a test case with that output?

@danielnelson danielnelson modified the milestones: 1.4.0, 1.4.1 Aug 23, 2017
Signed-off-by: Rickard von Essen <rickard.von.essen@gmail.com>
@rickard-von-essen
Copy link
Contributor Author

Added a test case. (Just had to get a FreeBSD vagrant box up.)

@danielnelson danielnelson merged commit 0502b65 into influxdata:master Aug 25, 2017
danielnelson pushed a commit that referenced this pull request Aug 25, 2017
@danielnelson danielnelson modified the milestones: 1.4.0, 1.4.1 Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants