datastax / adelphi Goto Github PK
View Code? Open in Web Editor NEWAutomation tool for testing C* OSS that assembles cassandra-diff, nosqlbench, fqltool
License: Apache License 2.0
Automation tool for testing C* OSS that assembles cassandra-diff, nosqlbench, fqltool
License: Apache License 2.0
NoSQLBench supports sending metrics directly to a graphite server: http://docs.nosqlbench.io/#/docs/reference/advanced_metrics.md
We can launch a Graphite server in the beginning of the workflow (or during the Helm chart installation) to collect the data generated by NSQLB.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-95
Current process for merging PRs created by "adelphi contribute" is a manual squash merge which doesn't do anything to clean up feature branches created on the users fork of our repo. We should consider automating an entire "merge submission" process which would:
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-66
Current work on anonymizer testing (#54) excludes custom indexes from anonymizer output under the assumption that custom index classes may or may not be present when the schema is recreated. This is certainly true as a general matter, but it is the case that some well-known custom index classes will always (or at least nearly always) be available in practice. This ticket intends to refine this logic a bit by allowing a defined set of well-known custom index classes to be included in the anonymized schema output.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-104
We rely on the C* Operator to handle the state of the clusters we start with cassandra-quality.
One of the requirements for this project is the following:
Automated cassandra-diff mixed version integration testing
command-line based
k8s, ds operator + sidecar orchestration
populate data in 1st cluster w/3 node RF=3 with arbitrary schema and nosqlbench
spin up 2nd cluster on same version
>>> upgrade 1 of the 3 nodes <<<
populate data in 2nd cluster
cassandra-diff to ensure both clusters match
But the Operator doesn't allow upgrading a single node.
This is currently a BLOCKER.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-97
The storage volume for the results should be configurable such that the user can choose the most appropriate settings depending on his env (cloud or local).
So far, we've had the following challenges:
What we should do instead is exposing the relevant storage settings in values.yaml (we already expose storageClass
) such that we can configure Adelphi to use AWS S3 or Google Cloud Storage.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-68
We need to create a setup.py
file for the anonymizer and distribute it as a package in pypi.org.
For some inspiration, we can follow the Python driver example: https://github.com/datastax/python-driver/blob/master/setup.py#L416
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-96
Add support for the concurrency level of the data integrity testing executed by Adelphi to be controlled by the user.
Allow for a number of concurrent threads to be provided via helm values.
N/A
Initial review of the Gemini command-line options looks like this can be controlled through the following flag:
-c, --concurrency uint Number of threads per table to run concurrently (default 10)
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-63
Issue #9 introduced a way to build the C* image from the source to use it in the target cluster and made it the default behavior, but instead we should give preference to using a publicly available image from Dockerhub, if possible.
The goal of this ticket is to add some conditional in the Argo workflow that checks the git_hash
configuration and, if not provided, skips building the image.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-89
While discussing a few ideas with @absurdfarce, he suggested that we allow uploading schemas straight from the anonymizer to make the process even more straightforward.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-91
Add support for the read/write ratio of the data integrity testing executed by Adelphi to be controlled by the user.
Allow for a read:write ratio to be provided via helm variables.
This could be specified as an integration from 0-100 or as a selection fo enumerated values which might make sense, like:
N/A
An initial look at Gemini indicates that this setting is not currently available on their side, to support it here we'd need to request/add support for it there.
Issue created in Gemini project: scylladb/gemini#262
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-64
Up until now, we've been building all the dependencies that have no official Docker images on the fly (Gemini, Spark, cassandra-diff) within the k8s cluster itself at every execution of Adelphi in order to have a consistent and simple experience locally or across cloud envs, but as we add more tools, the startup time adds up.
Instead, we should start by at least providing Dockerfiles such that one could build these images once and reuse them locally with k3d. Later, we could publish these images to a public registry.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-85
The PR we contributed to Gemini for supporting authentication has been merged and is available starting with v1.7.3.
We should simply download the binary and use that now, instead of building from the source.
┆Issue is synchronized with this Asana task by Unito
While talking with John Sanda, he let me know that in order to use a custom C* image in the Cass-Operator, we have to create a new sidecar image first because the C* version is hardcoded in the Dockerfile, for example: https://github.com/datastax/management-api-for-apache-cassandra/blob/master/Dockerfile-3_11
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-107
Now that we have a way to use the output of the schema anonymizer per #8, we should automate the execution of the script itself. That is, given the address of a prod C* (as a Helm config value) we should run the script as the first step of the workflow.
If the user provides the schema manually in values.yaml, that should take precedence over running it in prod, in other words, it should be a conditional step in the workflow.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-71
NoSQLBench and Gemini (and Harry in the future) all allow for setting an initial seed for random data generation.
We should externalize this as a Helm config prop such that we can record the used seed and reproduce a test run using it.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-94
Current work on the anonymizer test suite (see #54) is adding an integration test suite which spins up various versions of C* via Docker, loads them up with a complex schema and performs anonymization on the resulting keyspace(s). The driver for this effort is currently implemented as a shell script with results returned via the return value... but this only allows a binary fail/pass determination to be made.
Ideally we'd like to get individual feedback of success or failure for each C* version. A straightforward way to accomplish this would be to conver this runner to Python and bring in a well-known testing framework.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-105
The goal of this issue is to include some smoke tests that verify the basic Adelphi features.
We want to catch things like incompatibilities of running Adelphi or the Cass-Operator on the latest version of k3d.
We can also leverage this work to produce a routine/scheduled cadence of tests against C* trunk to externally verify its continued reliability over time.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-88
Currently, both source and target clusters are created as soon as the Helm chart is installed because we're using prebuilt C* images. But once we switch to custom C* images built from the source code, we have to wait for the image to be ready before trying to launch the cluster.
In other words, the installation of a CassandraCluster has to be a step in the Argo workflow.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-90
Some settings change depending on the environment (for example, the default storageClass for k3d is local-path
whereas for GKE it is standard
).
We can provide some sample config files that are tuned for the environment the user is targeting (local or cloud).
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-87
In some cases, helm seems to install workflows before workflow templates, which cause an invalid spec error.
┆Issue is synchronized with this Asana task by Unito
┆Issue is synchronized with this Asana task by Unito
Recent changes have moved the anonymization process to an earlier point in the workflow of the "adelphi" app. As a result the actual files in the generated Github PRs are named after anonymized keyspaces... which leaves a user no obvious way to determine if a keyspace they want to submit to Adelphi has already been submitted. This ticket would address this deficiency.
We likely wouldn't need much more than (1) adding a SHA hash of the (anonymized) schema as metadata and (2) allowing some kind of ability to query metadata from the "adelphi" app.
One might also ask why bother using the anonymized keyspace name for the file name in the repo if it doesn't provide any useful information. This question is entirely legit and is being explored elsewhere... it'll probably wind up in a ticket of it's own. That work should be considered to be separate from this effort, however. The focus of this ticket is very much on the ability to determine whether a schema has already been submitted. They're certainly related, however; I can imagine the naming convention in the repo being changed to use the SHA hash of the schema and this being the basis for the query mechanism.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-67
Per discussion in #34 (comment)
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-92
Documenting this here mainly so that there's a record of it. Fix will likely be implemented as part of the ongoing work for #67.
Observed while attempting to run "adelphi contribute" using Python 2.x after switching code to use the repo at datastax/adelphi-schemas (which is in the "datastax" organization):
Traceback (most recent call last):
File "/work/git/adelphi/python/setup/bin/adelphi", line 202, in <module>
export(obj={}, auto_envvar_prefix="ADELPHI")
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/core.py", line 829, in __call__
return self.main(*args, **kwargs)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/click/decorators.py", line 21, in new_func
return f(get_current_context(), *args, **kwargs)
File "/work/git/adelphi/python/setup/bin/adelphi", line 196, in contribute
fork_repo = origin_repo.create_fork()
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/github/Repository.py", line 2088, in create_fork
"POST", self.url + "/forks", input=post_parameters,
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/github/Requester.py", line 322, in requestJsonAndCheck
verb, url, parameters, headers, input, self.__customConnection(url)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/github/Requester.py", line 413, in requestJson
return self.__requestEncode(cnx, verb, url, parameters, headers, input, encode)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/github/Requester.py", line 470, in __requestEncode
requestHeaders["Content-Type"], encoded_input = encode(input)
File "/work/git/adelphi/python/setup/lib/python2.7/site-packages/github/Requester.py", line 411, in encode
return "application/json", json.dumps(input)
File "/usr/lib64/python2.7/json/__init__.py", line 244, in dumps
return _default_encoder.encode(obj)
File "/usr/lib64/python2.7/json/encoder.py", line 207, in encode
chunks = self.iterencode(o, _one_shot=True)
File "/usr/lib64/python2.7/json/encoder.py", line 270, in iterencode
return _iterencode(o, 0)
File "/usr/lib64/python2.7/json/encoder.py", line 184, in default
raise TypeError(repr(o) + " is not JSON serializable")
TypeError: NotSet is not JSON serializable
Underlying problem here is the Repository.create_fork function. That function in v1.45 of PyGithub is as follows:
def create_fork(self, organization=github.GithubObject.NotSet):
"""
:calls: `POST /repos/:owner/:repo/forks <http://developer.github.com/v3/repos/forks>`_
:param organization: string or "none" or "*"
:rtype: :class:`github.Repository.Repository`
"""
assert organization is github.GithubObject.NotSet or isinstance(
organization, (str, six.text_type)
), organization
post_parameters = {}
if organization is not None:
post_parameters["organization"] = organization
headers, data = self._requester.requestJsonAndCheck(
"POST", self.url + "/forks", input=post_parameters,
)
return Repository(self._requester, headers, data, completed=True)
Note the disconnect: the assertion is centered around a check for NotSet (which is the default value) but the val is added to post_parameters if the organization is not None. This results in the non-serializable NotSet being added to the outgoing HTTP request... which ultimately results in the TypeError shown above.
This bug was identified and fixed in PyGithub/PyGithub#1383. The problem is that (a) that fix wasn't rolled out until PyGithub 1.46 (see https://github.com/PyGithub/PyGithub/releases/tag/v1.46) and (b) the last version of PyGithub that supports Python 2.x is 1.45 (see https://github.com/PyGithub/PyGithub/releases/tag/v1.45).
Upshot is that if you use Python 2.x you'll get a version of PyGithub that includes this bug. If the user your Github token comes from is in an organization this becomes very problematic since you have to specify whether you want the fork created in the org or in your personal space. For extra fun no value of the organization param in the function below will accomplish that goal: if you pass None you'll fail the assertion but if you pass NotSet you'll hit the JSON serialization issue.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-62
At the moment, Adelphi operates on one schema at a time, as specified by the cql_schema
prop in values.yaml.
But with the addition of the schemas repo in #40, we have to run through multiple schemas now.
One possible approach is to implement a loop in the Argo workflow instead of resubmitting the helm chart multiple times to reduce the overhead.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-99
At the moment, we can simply tell whether the tests worked by having a succeeded workflow execution, but we should collect the results generated by cassandra-diff, Gemini and NoSQLBench and report them to the user.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-75
Current naming convention for schema files is /. After this was implemented we moved schema anonymization higher up in the process which means is always "ks_0" (or something similar) at the time of file creation. As a result we're very subject to collisions; the following should make it clear why:
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-65
#40 provided some initial support for detecting and working with virtual keyspaces in the export process but this work won't be completed in that ticket. Specifically we want to make sure the following things are true:
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-98
It's valuable to have some details about an anonymized schema that has been contributed to Adelphi, for example, purpose and maturity level.
We should allow for these command-line options such that the user can enter this information when running the script.
The values would be commented annotations in the beginning of the CQL file.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: AD-86
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.