Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement "kubedb get" #12

Merged
merged 5 commits into from
May 5, 2017
Merged

Implement "kubedb get" #12

merged 5 commits into from
May 5, 2017

Conversation

mirshahriar
Copy link
Contributor

No description provided.

@mirshahriar mirshahriar self-assigned this Apr 28, 2017
@mirshahriar mirshahriar requested review from tamalsaha and sadlil April 28, 2017 05:26
shahriar added 2 commits May 1, 2017 17:04
glide.yaml Outdated
@@ -4,3 +4,7 @@ import:
- package: github.com/appscode/log
- package: k8s.io/kubernetes
version: ~1.5.0
- package: github.com/spf13/cobra
version: 9c28e4bbd74e5c3ed7aacbc552b2cab7cfdfe744
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

version: master

glide.yaml Outdated
- package: github.com/spf13/cobra
version: 9c28e4bbd74e5c3ed7aacbc552b2cab7cfdfe744
- package: github.com/spf13/pflag
version: c7e63cf4530bcd3ba943729cee0efeff2ebea63f
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

version: master

@@ -0,0 +1,16 @@
package kube
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move to client.go

}

// New create a new Client
func GetKubeCmd(cmd *cobra.Command) *Client {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use Factory directly.

pkg/cmd/get.go Outdated
},
}

util.AddContextFlag(cmd)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a persistent flag

replicas: 1
serviceAccountName: governing-elasticsearch
version: 2.3.1-v2.3
status:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

status:
   DatabaseStatus: Running
   creationTime: 2017-04-27T04:46:52Z

This looks inconsistent.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DatabaseStatus should be lowe camel like other. Also, I think kube uses the word phase.

status:
   phase: Running
``

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@tamalsaha tamalsaha merged commit 8d7ada0 into master May 5, 2017
@tamalsaha tamalsaha deleted the cli branch May 5, 2017 07:16
tamalsaha added a commit that referenced this pull request Jul 8, 2018
5e13446a shfmt -l -w -ci -i 2 *.sh
499a1ab8 Add kubeware script
8e0811f8 Add kubevault script
f1cf7f24 Rename kiteci to kubeci
0e54dac9 Use go 1.9 compiler since tar package breaks static build
7117c1b8 Add soter image
a1aad737 Update README.md
eb754899 Mount /.cache folder to /tmp
dcf8ff35 Don't use upx with Windows binaries
f1197df8 Update README.md
6e94151a Use Golang 1.10.0-alpine
2bd987dd Add kubepack script
bf8075a7 Update Golang to 1.9.4
2634f8cc Keep the original file when bzip2-ing
f38a25e0 Use bzip2 without tar
21306b4c Fix upx command
17b0826f Fix compress process
068e6317 Conditionally enable upx
dd8f2507 Stop using upx
a40fdcb1 Use zip to compress in windows
7aea4cbb Update libbuild.py
59c194c2 Create *.tar.bz2 archives (#17)
53cabf10 Compress Go binaries using upx (#16)
81dd7359 only update canary tags for images built from master branch (#14)
c21c4816 Switch to GO 1.9.3
f0faa64b Remove `-it` from docker builds
430a90bc Use kiteci/golang:1.9.2-alpine image
fc9db855 Add kiteci.sh
55918b96 Add pharmer script
d8ec9c74 Update to go 1.9.2 alpine image
4c4d9152 Use Docker hub for public images
f1e269ab Update alpine image to 1.9.1
7b029988 Set GOBIN var
c409b02a Use current user to build alpine image.
b70065d4 Add build command for alpine (#12)
27220cfa Update libbuild.py

git-subtree-dir: hack/libbuild
git-subtree-split: 5e13446ab1012269f7c3ba914dbf93a399db41ed
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.

3 participants