kemko/nomad
1
0
Fork 0
mirror of https://github.com/kemko/nomad.git synced 2026-01-05 01:45:44 +03:00
Code Issues Packages Projects Releases Wiki Activity
13,858 Commits 1 Branch 0 Tags
e9a7978367fa6004e663a224ce0574d0814cebd2
Commit Graph

13858 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Danielle Tomlinson
e9a7978367 DROP: disable consul tests on windows 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
d81e632157 testutil: Start vault in the same routine as waiting 
This is a workaround for the windows process model.

Go os/exec does not pass the parent process handle to the child
processes STARTUPINFO struct, this means that unless we wait in
the _same_ execution context as Starting the process, the
handle will be lost, and we cannot kill it without regaining
a handle.

A better long term solution would be a higher level process
abstraction that uses windows.CreateProcess on windows.
 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
dcce2d7247 vaultclient: use require for error assertions 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
d33809b886 fingerprint: Limit vault shutdown waiting 
When vault is installed through chocolatey, it also installs a shim that
will not pass kill signals to the child. This means the process will
never actually terminate, and we lose the process handle.

Here, rather than waiting forever, we timeout fast.
 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
3078c24f79 vaultclient: Update tests for vault 1.0 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
b05d362eb1 fingerprinter: Use HCLogger for windows 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
a51aeb224f testtask: Build on windows 2019-01-17 18:43:13 +01:00
Danielle Tomlinson
eab62662de chore: Setup appveyor for windows test execution 2019-01-17 18:43:13 +01:00
Mahmood Ali
4ff1e16706 Merge pull request #5202 from hashicorp/b-no-gpl-gopsutil 
Update shirou/gopsutil dep to use non-GPL code
 2019-01-17 11:40:04 -05:00
Mahmood Ali
d6c3626d16 Update shirou/gopsutil dep to use non-GPL code 
Previously used `github.com/shirou/gopsutil`[1], used some GPL code [2].
This was somewhat unintentional, and was addressed later [3].

Due to being late in the cycle of Nomad release when this is noticed,
and time elapsed since we updated the dependency, we want to be
conservative in our package updates.

As such, we opted to go with forking the repo to use the previously used
version with the GPL removal code commit, done in [4].

[1] 5776ff9c7c
[2] 5776ff9c7c/host/include/smc.c
[3] c95755e4bc
[4] 62d5761ddb
 2019-01-17 11:08:30 -05:00
Mahmood Ali
3c9de20820 fix imports 2019-01-16 19:53:48 -05:00
Michael Schurter
10068422e0 Merge pull request #5197 from hashicorp/b-rkt-cpu 
rkt: revert to pre-0.9 --cpu flag
 2019-01-16 15:05:21 -08:00
Preetha
dab186a4a2 Merge pull request #5182 from hashicorp/b-docker-fingerprint-log-once 
Make driver logging less redundant
 2019-01-16 11:35:33 -06:00
Preetha Appan
3e16b52361 clean up read access 2019-01-16 11:04:11 -06:00
Danielle Tomlinson
435a8bae96 Merge pull request #5193 from hashicorp/dani/logmon-reattach 
logmon: Reattach to existing loggers
 2019-01-16 17:34:13 +01:00
Preetha Appan
80c00fc268 Refactor logging in drivers to use a tri-state boolean 
Changes logging warnings/errors only if the state changes
from healthy to unhealthy
 2019-01-16 10:19:31 -06:00
Preetha Appan
35ab26658c Make docker driver logging less redundant 2019-01-16 10:16:57 -06:00
Michael Schurter
4dc1ce02cd Merge pull request #5199 from hashicorp/test-dispatch 
test: porting TestTaskRunner_SimpleRun_Dispatch
 2019-01-16 06:33:51 -08:00
Danielle Tomlinson
828d5f5a53 logmon: Reattach to existing loggers 
This commit prevents us from creating duplicate logmon hooks when
restoring allocations by persisting the logmon reattach config using
HookData.
 2019-01-16 14:56:10 +01:00
Michael Schurter
152ba08207 test: porting TestTaskRunner_SimpleRun_Dispatch 
Porting test from 0.8 to 0.9.
 2019-01-15 15:22:13 -08:00
Alex Dadgar
8e9fefcb94 Merge pull request #5046 from oleksii-shyman/docker_runtimes 
Nvidia runtime
 2019-01-15 14:34:14 -08:00
Michael Schurter
97be19b176 rkt: revert to pre-0.9 --cpu flag 
See
https://github.com/hashicorp/nomad/issues/3394#issuecomment-453296121
for details. During 0.9 development we switched to shares, but we'd
prefer to maintain backward compat.
 2019-01-15 13:15:28 -08:00
Chris Baker
bb57f00f2c Merge pull request #5194 from hashicorp/f-minor-update-docs-tense 
docs: updated tense around fixed bug
 2019-01-15 14:50:57 -05:00
oleksii.shyman
cc98f282d4 Add support for docker runtimes 
- docker fingerprint issues a docker api system info call to get the
  list of supported OCI runtimes.
  - OCI runtimes are reported as comma separated list of names
  - docker driver is aware of GPU runtime presence
  - docker driver throws an error when user tries to run container with
  GPU, when GPU runtime is not present
  - docker GPU runtime name is configurable
 2019-01-15 11:34:47 -08:00
oleksii.shyman
943b375b72 Update go-dockerclient and docker dependencies 
- go-dockerclient is updated because of lack of "runtime" API field
 - docker/docker is updated because go-dockerclient depends on docker
 master branch
 2019-01-15 11:28:57 -08:00
Alex Dadgar
48283ae752 Merge pull request #5195 from hashicorp/d-update-087-date 
make change to the 0.8.7 release date
 2019-01-15 09:53:07 -08:00
Cathy Chang
1e7b110962 make change to the 0.8.7 release date 2019-01-15 09:47:13 -08:00
Chris Baker
25013af5b6 docs: updated tense around fixed bug 2019-01-15 17:43:03 +00:00
Danielle Tomlinson
9cf6e1ae27 Merge pull request #5192 from hashicorp/dani/executor-close 
executor: Always close stdout/stderr fifos
 2019-01-15 17:49:04 +01:00
Danielle Tomlinson
fb1e24d2b9 Merge pull request #5184 from hashicorp/dani/b-logmon-reattach 
docker: Terminate dockerlogger
 2019-01-15 16:48:40 +01:00
Danielle Tomlinson
17dbace46b executor: Always close stdout/stderr fifos 2019-01-15 16:47:27 +01:00
Michael Schurter
3a491ab2b4 Merge pull request #5187 from hashicorp/test-consul 
Port a bunch of pre-0.9 Consul tests to 0.9
 2019-01-15 07:41:50 -08:00
Mahmood Ali
1bfec9e8e7 Merge pull request #5189 from hashicorp/b-fix-executor-logs 
propagate logs to executor plugin
 2019-01-15 10:26:01 -05:00
Danielle Tomlinson
f9a4594095 docker: Terminate dockerlogger 
Previously, we did not attempt to stop Docker Logger processes until
DestroyTask, which means that under many circumstances, we will never
successfully close the plugin client.

This commit terminates the plugin process when `run` terminates, or when
`DestroyTask` is called.

Steps to repro:

```
$ nomad agent -dev
$ nomad init
$ nomad run example.nomad
$ nomad stop example
$ ps aux | grep nomad # See docker logger process running
$ signal the dev agent
$ ps aux | grep nomad # See docker logger process running
```
 2019-01-15 14:58:05 +01:00
Mahmood Ali
2cc4466573 propogate logs to executor plugin 2019-01-15 08:25:03 -05:00
Alex Dadgar
109c5ef650 Merge pull request #5173 from hashicorp/b-log-levels 
Plugins use parent loggers
 2019-01-14 16:14:30 -08:00
Nick Ethier
97a73e0116 Merge pull request #5018 from hashicorp/f-executor-stats 
executor: streaming stats api
 2019-01-14 15:02:35 -05:00
Michael Schurter
7445e418ca test: port some pre-0.9 DeploymentHealth tests 
Skipping a failing one as I need to move to some other work and don't
want to leave this work orphaned on my machine.
 2019-01-14 09:56:53 -08:00
Michael Schurter
b58a77ecc0 test: assert service interpolation behavior 
Ported from pre-0.9 tests.
 2019-01-14 09:56:53 -08:00
Michael Schurter
df0a6dc34e test: add some extra logging 2019-01-14 09:56:53 -08:00
Michael Schurter
ceee126241 Remove old comment; it's been fixed! 2019-01-14 09:56:53 -08:00
Michael Schurter
99a5586aed test: assert shutdown delay deregs first 
Restore a pre-0.9 test that asserts Consul services are deregistered
before a task's shutdown delay.
 2019-01-14 09:56:53 -08:00
Michael Schurter
903779769d Update client/allocrunner/taskrunner/stats_hook.go 
Co-Authored-By: nickethier <ncethier@gmail.com>
 2019-01-14 12:31:27 -05:00
Nick Ethier
091cdbcb12 tr: stop stats collection on Exited hook 2019-01-14 12:30:14 -05:00
Nick Ethier
d086bb2d20 qemu: missing gofmt 2019-01-13 16:06:56 -05:00
Michael Schurter
60acfc8289 Update drivers/qemu/driver_test.go 
use t.Logf instead of fmt.Printf

Co-Authored-By: nickethier <ncethier@gmail.com>
 2019-01-12 21:33:55 -05:00
Nick Ethier
fc84e8f2bc tr: add retry /w backoff to stats_hook failure 2019-01-12 12:18:24 -05:00
Nick Ethier
72a4685534 drivers: plumb grpc client logger 2019-01-12 12:18:23 -05:00
Nick Ethier
9904463da2 executor: fix failing stats related test 2019-01-12 12:18:23 -05:00
Nick Ethier
f6af1d4d04 docker: add test for stats collection 2019-01-12 12:18:22 -05:00