Skip to content
Projects
Groups
Snippets
Help
Loading...
Sign in
Toggle navigation
H
helm3
Project
Project
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Commits
Issue Boards
Open sidebar
go
helm3
Commits
e28e2419
Commit
e28e2419
authored
Sep 12, 2016
by
fibonacci1729
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
fix(tiller): re-enable skipped test, golint, & gofmt all the things
parent
79e57720
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
4 additions
and
6 deletions
+4
-6
release_server.go
cmd/tiller/release_server.go
+3
-3
release_server_test.go
cmd/tiller/release_server_test.go
+1
-3
No files found.
cmd/tiller/release_server.go
View file @
e28e2419
...
@@ -225,10 +225,10 @@ func (s *releaseServer) GetReleaseContent(c ctx.Context, req *services.GetReleas
...
@@ -225,10 +225,10 @@ func (s *releaseServer) GetReleaseContent(c ctx.Context, req *services.GetReleas
if
req
.
Version
<=
0
{
if
req
.
Version
<=
0
{
rel
,
err
:=
s
.
env
.
Releases
.
Deployed
(
req
.
Name
)
rel
,
err
:=
s
.
env
.
Releases
.
Deployed
(
req
.
Name
)
return
&
services
.
GetReleaseContentResponse
{
Release
:
rel
},
err
return
&
services
.
GetReleaseContentResponse
{
Release
:
rel
},
err
}
else
{
rel
,
err
:=
s
.
env
.
Releases
.
Get
(
req
.
Name
,
req
.
Version
)
return
&
services
.
GetReleaseContentResponse
{
Release
:
rel
},
err
}
}
rel
,
err
:=
s
.
env
.
Releases
.
Get
(
req
.
Name
,
req
.
Version
)
return
&
services
.
GetReleaseContentResponse
{
Release
:
rel
},
err
}
}
func
(
s
*
releaseServer
)
UpdateRelease
(
c
ctx
.
Context
,
req
*
services
.
UpdateReleaseRequest
)
(
*
services
.
UpdateReleaseResponse
,
error
)
{
func
(
s
*
releaseServer
)
UpdateRelease
(
c
ctx
.
Context
,
req
*
services
.
UpdateReleaseRequest
)
(
*
services
.
UpdateReleaseResponse
,
error
)
{
...
...
cmd/tiller/release_server_test.go
View file @
e28e2419
...
@@ -646,8 +646,6 @@ func TestUninstallPurgeRelease(t *testing.T) {
...
@@ -646,8 +646,6 @@ func TestUninstallPurgeRelease(t *testing.T) {
}
}
func
TestUninstallPurgeDeleteRelease
(
t
*
testing
.
T
)
{
func
TestUninstallPurgeDeleteRelease
(
t
*
testing
.
T
)
{
t
.
Skip
(
"TestUninstallPurgeDeleteRelease"
)
c
:=
context
.
Background
()
c
:=
context
.
Background
()
rs
:=
rsFixture
()
rs
:=
rsFixture
()
rs
.
env
.
Releases
.
Create
(
releaseStub
())
rs
.
env
.
Releases
.
Create
(
releaseStub
())
...
@@ -667,7 +665,7 @@ func TestUninstallPurgeDeleteRelease(t *testing.T) {
...
@@ -667,7 +665,7 @@ func TestUninstallPurgeDeleteRelease(t *testing.T) {
}
}
_
,
err2
:=
rs
.
UninstallRelease
(
c
,
req2
)
_
,
err2
:=
rs
.
UninstallRelease
(
c
,
req2
)
if
err2
!=
nil
{
if
err2
!=
nil
&&
err2
.
Error
()
!=
"'angry-panda' has no deployed releases"
{
t
.
Errorf
(
"Failed uninstall: %s"
,
err2
)
t
.
Errorf
(
"Failed uninstall: %s"
,
err2
)
}
}
}
}
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment