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
Refactor type Environment
and env Config
to support new typeklient.Client
#34
Refactor type Environment
and env Config
to support new typeklient.Client
#34
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vladimirvivien The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
klient.Client
klient.Client
Environment
to support new typeklient.Client
Environment
to support new typeklient.Client
Environment
and env Config
to support new typeklient.Client
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, lgtm, just need to fix the lint issues we I think we are good to go
} | ||
|
||
func randNS() string { | ||
bytes := make([]byte, 25) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can use github.com/google/uuid
and generate an uuid and remove the testns-%s
.
just an idea :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cpanato thanks. I will checkout the google project.
Also, I added the "test" so that if, for whatever reasons, a namespace lingers, you will be able to easily identify it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make sense, as i said just a suggestion :)
…lient This PR refactors type env.Environment and envconf.Config to support new type klient.Client. Thype klient.Client makes easy to access the functionalities of the klient package from one type withouth having to create several steps. Signed-off-by: Vladimir Vivien <vivienv@vmware.com>
b394c9f
to
15c3022
Compare
/lgtm |
This PR is needed to refactor the way type
Environment
is constructed and initialized with packageklient
which did not exist when this package was created. This work is also a precursor to the refactoring work that will take place to fix context propagation.This potentially will address
Context propagation will be handled in a future PR