bwsw / cs-vault-server Goto Github PK
View Code? Open in Web Editor NEWCloudStack Vault Plugin Server Handler
License: Apache License 2.0
CloudStack Vault Plugin Server Handler
License: Apache License 2.0
You defined the scope: Zookeeper, but inside the scope you use zooKeeperRootNode
, why not to use just rootNode
. Change everywhere to be logically motivited. Environment variables should be built like <SCOPE>_<VAR>_<NAME>
, e.g. ZOOKEEPER_ROOT_NODE
.
Change everywhere!
"Tags were got" is a wrong phrase, use something like "Tags retrieved". I don't think that passive voice in the past are best way to express such great outcome. If you DO debug, why don't you debug actual TAGS that retrieved. What is the case to use such useless debug information??? FIX everywhere.
scala> 'с' == 'c'
res1: Boolean = false
Right now it uses: vault.ro, vault.rw, but we should be able to configure prefix, like [csext.]vault.ro
to reflect external requirements. It should be configured in an config variable like:
TAGNAME_PREFIX=csext
"." (dot) symbol should be appended automatically if trailing dot is missing. By default TAGNAME_PREFIX is just empty.
Which components should be replaced by components from others projects.
Try to divide into tasks.
Accounts and VMs reside in domains, when you request something domain must be specified because without the domain you implicitly work in root domain (ask Elena Ershova if required about domains).
Two accounts with the same names might exist in different domains, e.g. root domain and room-1 domain.
a) Make a test plan
b) Prepare mocks
c) Implement test's cases
May be move even more and add "UUID"? Remove it.
If there is a reason, comment and describe why it's required.
Add vault.path to place also a keyspace (host, prefix) which is affected by ro/rw keys both to user and vm. See #29 first as it's affected too.
a) Implement negative tests for services
b) Implement negative and positive tests for taskCreators of services
a) Implement the creation and receive of a Token using the vault scala library
Create the application architecture and describe it through UML class diagram.
Why do token period reside inside "app" scope if it relates to vault scope? Fix it.
I believe Idea should warn you against use of boolean constant as a in-place argument. Either do
val meaningfulName = true
fun(meaningfulName)
or
fun(funArgument = true)
because tags are unique, List is error prone. Use set.
see #83
tokenPeriod = ${?TOKEN_PERIOD}
tokenPeriod = "3562" //10 years
If the param allows adding more than one server, then the name should be list. Basically, I beliave all where we use list, we should use endpoints, if use one - endpoint
So
Zookeeper {
endpoints = "zk://127.0.0.1:2181/"
endpoints = ${ZOOKEEPER_ENDPOINTS}
}
looks fine. Change everywhere. Change environment variables.
getVmTagsById
getAccountIdByVmId
getVmTags
getVmOwnerAccount
Why it's "var" not "val" btw. Why the name is "threadLocalClientList" is it means something, do you think "clientList" is a bad name for it?
a) Implement the request in ACS to receive all users and their tags for the account
b) Implement the creation of a tags in ACS for token saving
What is the purpose to accept by
class ConsumerManager(topic: String, brokers: String, components: Components)
topic, brokers, components parameters. I don't see the purpose. You are mixing levels of abstraction. Fix it.
Sometimes "StrictLogging" trait is used, sometimes just
private val logger = LoggerFactory.getLogger(this.getClass)
Is it for a reason? Unify. Don't do the mess.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.