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

feat: migrate keyspace if it doesn't exist. #74

Merged
merged 1 commit into from
Mar 18, 2015

Conversation

thibaultcha
Copy link
Member

Proposed solution to #68:

Instead of doing this in the bash script, we do it in main.lua's init_by_lua since the error handling from bin/kong is already handled.

We only run migrations if we detect than they were never ran before. We won't automatically run any new migration because the process shouldn't be transparent, users should know about it and backup their data if they want to.

  • migrate kong on first run
  • show nice error messages from the CLI when startup failed (when migration failed or when preparation failed since they are both done in init_by_lua)

@thibaultcha thibaultcha force-pushed the feature/startup-migration branch from 69bb42a to b0a9fb3 Compare March 12, 2015 06:11
@coveralls
Copy link

Coverage Status

Coverage increased (+0.38%) to 76.23% when pulling b0a9fb3 on feature/startup-migration into 9a5ae89 on master.

@thibaultcha thibaultcha force-pushed the feature/startup-migration branch 2 times, most recently from f6fb8c9 to 1190034 Compare March 18, 2015 02:02
+ test for startup migration

feat: CLI outputs nicer startup error messages

- Nicer error messages in stdout from init failure
- cleanup bin/kong

refactor: structure

- core renamed 'resolver'
- 'resolver' doesn't inherit from Base_plugin anymore (it is not a
  plugin)
- 'resolver' has nicer method names make it clearer to understand what
  is going on in nginx.conf
- kong code is not nested in src/kong anymore, rather directly put under
  src/
@thibaultcha thibaultcha force-pushed the feature/startup-migration branch from 1190034 to 031ec56 Compare March 18, 2015 02:09
thibaultcha added a commit that referenced this pull request Mar 18, 2015
feat: migrate keyspace if it doesn't exist.
@thibaultcha thibaultcha merged commit b576b0c into master Mar 18, 2015
@thibaultcha thibaultcha deleted the feature/startup-migration branch March 18, 2015 02:15
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.16%) to 73.56% when pulling 031ec56 on feature/startup-migration into 3d59f53 on master.

ctranxuan pushed a commit to streamdataio/kong that referenced this pull request Aug 25, 2015
feat: migrate keyspace if it doesn't exist.
bungle added a commit that referenced this pull request Oct 19, 2022
### Summary

### [0.8.13] - 2022-10-14
#### bug fixes
- **x509.\*:** fix set_extension will fail when a extension with same NID is not exist yet ([#75](fffonion/lua-resty-openssl#75)) [b2f57b8](fffonion/lua-resty-openssl@b2f57b8)

#### features
- **x509.altname:** support set and get IP addresses ([#74](fffonion/lua-resty-openssl#74)) [363c80d](fffonion/lua-resty-openssl@363c80d)
- **x509.store:** add set_flags ([#77](fffonion/lua-resty-openssl#77)) [8f3f16a](fffonion/lua-resty-openssl@8f3f16a)

### [0.8.11] - 2022-10-12
#### performance improvements
- **\*:** reuse cdata to improve performance [fc9cecd](fffonion/lua-resty-openssl@fc9cecd)
bungle added a commit that referenced this pull request Oct 19, 2022
### Summary

### [0.8.13] - 2022-10-14
#### bug fixes
- **x509.\*:** fix set_extension will fail when a extension with same NID is not exist yet ([#75](fffonion/lua-resty-openssl#75)) [b2f57b8](fffonion/lua-resty-openssl@b2f57b8)

#### features
- **x509.altname:** support set and get IP addresses ([#74](fffonion/lua-resty-openssl#74)) [363c80d](fffonion/lua-resty-openssl@363c80d)
- **x509.store:** add set_flags ([#77](fffonion/lua-resty-openssl#77)) [8f3f16a](fffonion/lua-resty-openssl@8f3f16a)

### [0.8.11] - 2022-10-12
#### performance improvements
- **\*:** reuse cdata to improve performance [fc9cecd](fffonion/lua-resty-openssl@fc9cecd)
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.

2 participants