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

CLI arguments to configure Cassandra datastore #151

Closed
5 tasks
subnetmarco opened this issue Apr 21, 2015 · 4 comments
Closed
5 tasks

CLI arguments to configure Cassandra datastore #151

subnetmarco opened this issue Apr 21, 2015 · 4 comments
Assignees

Comments

@subnetmarco
Copy link
Member

Implementing the following arguments in the CLI to override the kong.yml datastore configuration:

  • --cassandra-hosts
  • --cassandra-port
  • --cassandra-keyspace
  • --cassandra-timeout
  • --cassandra-keepalive
@subnetmarco
Copy link
Member Author

Maybe this issue should be renamed to "Allow to override any configuration property through CLI arguments" (not just cassandra properties)

@thibaultcha
Copy link
Member

I don't like the idea. it makes things complicated for nothing. Updating your config will be faster than writing a command with 20 char arguments.

@subnetmarco
Copy link
Member Author

Sometimes the config is generated automatically, or needs to be edited by an application, so the only option to update it now is by using regex and sed, which is not easy. Appending CLI arguments provides a way for better automation.

@subnetmarco subnetmarco added the dao label May 1, 2015
@thibaultcha
Copy link
Member

Don't see that happening, way too much of a hassle, way too many (and too long) arguments for that to be included in the CLI.

hutchic pushed a commit that referenced this issue Jun 10, 2022
### Summary

#### libyaml 0.2.2 release

- #95 -- build: do not install config.h
- #97 -- appveyor.yml: fix Release build
- #103 -- Remove unused code in yaml_document_delete
- #104 -- Allow colons in plain scalars inside flow collections
- #109 -- Fix comparison in tests/run-emitter.c
- #117 -- Fix typo error
- #119 -- The closing single quote needs to be indented...
- #121 -- fix token name typos in comments
- #122 -- Revert removing of open_ended after top level plain scalar
- #125 -- Cherry-picks from PR 27
- #135 -- Windows/C89 compatibility
- #136 -- allow override of Windows static lib name

#### libyaml 0.2.3 release

- #130 Fixed typo.
- #144 Fix typo in comment
- #140 Use pointer to const for strings that aren't/shouldn't be modified
- #128 Squash a couple of warnings in example-deconstructor-alt
- #151 Fix spelling for error message
- #161 Make appveyor config be a hidden file
- #159 Add CHANGES file
- #160 Always output document end before directive (YAML 1.2 compatibility)
- #162 Output document end marker after open ended scalars
- #157 change cmake target name from libOFF.a to libyaml.a
- #155 include/yaml.h: fix comments
- #169 Fixed missing token in example
- #127 Avoid recursion in the document loader.
- #172 Support %YAML 1.2 directives
- #66 Change dllexport controlling macro to use _WIN32
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

No branches or pull requests

2 participants