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

Allow named volume mounts in regex #4311

Merged

Conversation

nisseknudsen
Copy link
Contributor

Note: If this PR is just doc changes, please put [ci skip] in the body that way tests do not run.

  • Allows the usage of named volumes in storage:mount by validating correctly in Regex
  • Embeds Docker rules on named volumes:
  • Min 2 characters long in name
  • Needs to follow schema [a-zA-Z0-9][a-zA-Z0-9_.-]

@josegonzalez
Copy link
Member

@nisseknudsen mind adding a test case for this?

@josegonzalez
Copy link
Member

@nisseknudsen minor bump. I'm also happy to write the tests for you if you need that :)

@nisseknudsen
Copy link
Contributor Author

@josegonzalez : Totally - just got out of work for the week. I will check it rn - I hope I can directly understand how and where to write tests.

@josegonzalez josegonzalez changed the title Pr/allow named volume mounts in regex Allow named volume mounts in regex Jan 15, 2021
@josegonzalez josegonzalez merged commit c179958 into dokku:master Jan 15, 2021
@nisseknudsen nisseknudsen deleted the pr/allow_named_volume_mounts_in_regex branch January 15, 2021 23:00
josegonzalez pushed a commit that referenced this pull request Jan 18, 2021
# History

## 0.22.9

Install/update via the bootstrap script:

```shell
wget https://raw.githubusercontent.com/dokku/dokku/v0.22.9/bootstrap.sh
sudo DOKKU_TAG=v0.22.9 bash bootstrap.sh
```

### Bug Fixes

- #4334: @josegonzalez allow git pushes to apps that already exist but have old names
- #4335: @josegonzalez Add error checking to plugin:install and plugin:update
- #4338: @josegonzalez Strip trailing slashes in hostname from web installer
- #4332: @josegonzalez Correct aws release check in release-plugin script
- #4311: @nisseknudsen Allow named volume mounts in regex
- #4320: @josegonzalez Hardcode cnb workdir to /workspace

### New Features

- #4321: @josegonzalez Add git version to output

### Refactors

- #4329: @josegonzalez Standardize apt-get usage

### Documentation

- #4323: @stianlik Fix typo in nginx documentation

### Tests

- #4331: @josegonzalez Switch to upstream bats-core
- #4330: @josegonzalez Switch to using uuidgen for unique app names

### Other

- #4325: @WaviestBalloon Updated bootstrap.sh to not show when $DOKKU_TAG equals nothing
- #4324: @dependabot-preview[bot] chore(deps): bump socket.io from 3.0.5 to 3.1.0 in /tests/apps/.websocket.disabled
- #4319: @josegonzalez Update ruby dependencies in test app
- #4312: @dependabot-preview[bot] chore(deps-dev): bump heroku/heroku-buildpack-php from 187 to 188 in /tests/apps/php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants