MAAS

Merge ~troyanov/maas:add-go-test-dependency into maas:master

  1. Git
  2. lp:~troyanov/maas
  3. add-go-test-dependency
  4. Merge into master
Proposed by Anton Troyanov
Status: Merged
Approved by: Adam Collard
Approved revision: bc3a72a4b2a84e44c678cf1a360d94ea1ef8390b
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~troyanov/maas:add-go-test-dependency
Merge into: maas:master
Diff against target: 13 lines (+1/-1)
1 file modified
src/maasagent/Makefile (+1/-1)
Reviewer Review Type Date Requested Status
MAAS Lander Needs Fixing
Alberto Donato (community) Approve
Review via email: mp+444573@code.launchpad.net

Commit message

fix(ci): vendor as a prerequisite for test

To post a comment you must log in.
Revision history for this message
Alberto Donato (ack) wrote :

+1

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b add-go-test-dependency lp:~troyanov/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/2716/console
COMMIT: bc3a72a4b2a84e44c678cf1a360d94ea1ef8390b

review: Needs Fixing
Revision history for this message
Anton Troyanov (troyanov) wrote :

jenkins: !test

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasagent/Makefile b/src/maasagent/Makefile
2index 66c69ab..63aa91c 100644
3--- a/src/maasagent/Makefile
4+++ b/src/maasagent/Makefile
5@@ -32,7 +32,7 @@ install: build
6 install -t $(DESTDIR)/bin -D $(BUILD_DIR)/*
7
8 .PHONY: test
9-test:
10+test: vendor
11 CGO_ENABLED=1 $(GO) test -race ./...
12
13 .PHONY: generate

Subscribers

People subscribed via source and target branches