push eventdocker/app
commit sha d2c567b4d3a896d9a0de0c2bff0b5c11915a268a
Add warnings for env_file entries not copied As 'env_file' entries are not taken in account, we warn the user to copy that and fix it's compose file. Signed-off-by: Ulysses Souza <[email protected]>
commit sha c7927e1906d8637cd044be908b7eeb73720821ec
Merge pull request #610 from ulyssessouza/add-env_file-warning Add warnings for env_file entries not copied
push time in 5 days
PR merged docker/app
As 'env_file' entries are not taken in account, we warn the user to copy that and fix it's compose file.
- What I did
Add a warning for each env_file entry on each service
- How I did it
By checking the env_file block for each service entry in docker app init
- How to verify it
Create a docker-compose.yml file with:
version: '3.7'
services:
front:
image: "nginx"
env_file:
- "myenv.env1"
- "myenv.env2"
Run:
docker app init --compose-file docker-compose.yml myapp
- Description for the changelog
Add warnings for env_file entries not copied
- A picture of a cute animal (not mandatory but encouraged)

pr closed time in 5 days
push eventchris-crone/app
commit sha 88fcef1b2094b07ef42130c69a10d02674cb67a3
Download Docker CLI instead of building it Download the macOS and Linux Docker CLI from download.docker.com instead of building them. Signed-off-by: Christopher Crone <[email protected]>
push time in 5 days
pull request commentdocker/app
[rfd] Use dockerCli.Out() / Err()
I think we've actually implemented this in the time since you opened this PR @thaJeztah :) Thanks for the suggestion!
I'll close this and we'll open other issues if we find places we forgot to do this.
comment created time in 5 days
PR closed docker/app
Quick 5-minute write-up;
This should likely not make a difference for most situations, but try to keep a single source of truth where to print output
There's still some occurences of os.Stdout to detect if a terminal is attached, so perhaps we need a central solution for that
pr closed time in 5 days
push eventchris-crone/app
commit sha bdbcf4d0a5e482a940e1100606f0ef407b4fbc30
Add note to enable BuildKit to build in docker Adds a note in the "Build using Docker" section of BUILDING.md to explain that BuildKit is required and how to enable it. Signed-off-by: Nick Adcock <[email protected]>
commit sha dc83704d7f3c3538e6762f5ae907ee617b453d33
Update cnab-to-oci to v0.3.0-beta1 and cnab schema version to 1.0.0 Allow to push all images from the push/fixup command Signed-off-by: Yves Brissaud <[email protected]>
commit sha c9a62caae9af6ffc053c176ef863ab8f1ff2e4ce
Bump cnab-go and others Signed-off-by: Yves Brissaud <[email protected]>
commit sha afe63d7ce651861346174b6ad651c04ad5e4f0b0
Enable AutoBundleUpdate from cnab-to-oci Signed-off-by: Yves Brissaud <[email protected]>
commit sha 7c4126b1577aa02d71949bcc70fb48d40088994d
Enable push of images from cnab-to-oci Signed-off-by: Yves Brissaud <[email protected]>
commit sha 447b9e250f157287e2ce9aa8e9372c55eae57ddc
Remove platform flag for push Signed-off-by: Yves Brissaud <[email protected]>
commit sha f2408b6d08021059c894ef5da17457c5888e8844
Remove tag flag for push Signed-off-by: Yves Brissaud <[email protected]>
commit sha 4eba2d7938dea8026e617d6cf20498f9b333c532
Fix compose file in e2e test Only visible when trying to install it but still an issue Signed-off-by: Yves Brissaud <[email protected]>
commit sha 303135afd0f197bbc09569390a168d353f38f43e
Move and remove e2e methods Signed-off-by: Yves Brissaud <[email protected]>
commit sha de787f11db21f6af4b59801347bf4c4fe118a349
Fix various modification in e2e tests By example we do not display the full reference anymore but use relocation map Signed-off-by: Yves Brissaud <[email protected]>
commit sha 42218d2f214f26177b64d872dfe7e45224669fb1
cnab-go OperationConfigs panic if we pass a nil function Signed-off-by: Yves Brissaud <[email protected]>
commit sha d48d5cc36ca826b183f33d4569b4afd2ad422d27
save relocation map from cnab-to-oci When a relocation map is generated from `cnab-to-oci` save it within the same directory as the `bundle.json` file. More information about reloaction map can be found in corresponding `cnab-to-oci` issue: https://github.com/docker/cnab-to-oci/issues/58 The `bundle.Bundle` struct is now wrapped in a `relocated.Bundle` that will also contain the relocation map. Methods to fetch `bundle.json` and `relocation-map.json` as well as en entire bundle with the relocation map at once are moved to a `fetch` package to avoid dependency cycle. Signed-off-by: Yves Brissaud <[email protected]>
commit sha d0cc6fb0a8233af6d9c382b6b557c54e3f9f399e
apply relocation map when creating the installation Allow to replace local images by the one from the registry using stored `relocation-map.json` when creating the installation. The `bundle.json` is not modified. Signed-off-by: Yves Brissaud <[email protected]>
commit sha 0f5dd8a5623bbd7e7739abeaa445ac44fc8bfae0
fix coverage issues Be sure to remove the non needed lines. Signed-off-by: Yves Brissaud <[email protected]>
commit sha bff05c35001c6c7c46d0d90662e7c1a57181f006
Merge pull request #729 from zappy-shu/enable-buildkit-to-build-with-docker Add note to enable BuildKit to build in docker
commit sha 44932b629b0541b28d0898c5dfac6a5f978d1ba1
Merge pull request #719 from eunomie/relocation-push Push local docker images and use relocation map
commit sha c2ed96b365d76f6a157608dfa4c2906981e34b3c
Download Docker CLI instead of building it Download the macOS and Linux Docker CLI from download.docker.com instead of building them. Signed-off-by: Christopher Crone <[email protected]>
push time in 5 days
push eventchris-crone/app
commit sha 0fe1aaf45452efa60fe66dbd710d6cf8b290c180
Remove target-context flags Signed-off-by: Ulysses Souza <[email protected]> Signed-off-by: Nicolas De Loof <[email protected]>
commit sha f9acea2cf48c2588750e99feae7f73ca70165acc
Introduce --installer-context option Create a new DockerCli used to run installer image within requested context cli.Cli (not DockerCli implementation) doesn't offer any way to switch contexts once intialized. Signed-off-by: Nicolas De Loof <[email protected]>
commit sha 020761c25da09c5a15f4001a85ed300197c0ea3c
We can't access DinD by `localhost` from within the installer image, this would only resolve to container's loopback so we use computer external IP Signed-off-by: Nicolas De Loof <[email protected]>
commit sha d10e2f5c88faaccd01ff303b5f9e12d34608fbc3
Use of registryAdress makes tests flacky as tabwritter will change formatting depending on number of digits in IP address 10.100.97.67:32795/c-myapp:latest 10.100.102.144:32795/c-myapp:latest Signed-off-by: Nicolas De Loof <[email protected]>
commit sha 82878c7a8aeba517d146c37614217e3e5e03ee0e
Use ChooseHostInterface from kube to select interface We need to select interface used to access default gateway, as docker0 bridge or Windows DockerNAT won't route traffic to containers by exposed port Signed-off-by: Nicolas De Loof <[email protected]>
commit sha c1a821753d6c48f467502048efc9e2365303f354
Fix rebase issues Signed-off-by: Nicolas De Loof <[email protected]>
commit sha 8beb2f139853113b3389cc6eeb75ce7d6b27997e
Discover the IP address to based on default gateway Without this check, Linux bridge docker0 or Winwdows DockerNAT could be selected as "host IP" but won't allow to route to container Same issue probalby exists on Linux with docker0 bridge Here we discover the default gateway, and search for the interface configured on the same network. Signed-off-by: Nicolas De Loof <[email protected]>
commit sha a850314fdb21ba957de48af7042adc82c5dce36c
Document new --installer-context option Signed-off-by: Nicolas De Loof <[email protected]>
commit sha 1effbfb009ed60c246526bcb4238567e7486147e
Download Docker CLI instead of building it Download the macOS and Linux Docker CLI from download.docker.com instead of building them. Signed-off-by: Christopher Crone <[email protected]>
push time in 5 days
PR opened docker/app
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/docker/app/blob/master/CONTRIBUTING.md
** Make sure all your commits include a signature generated with git commit -s **
For additional information on our contributing process, read our contributing guide https://docs.docker.com/opensource/code/
If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx"
Please provide the following information: -->
- What I did Download the CLI for macOS and Linux as they are available on download.docker.com. This should make the build slightly faster.
Note the CLI is only used for testing – we do not ship it.
- How I did it Replace build with curl.
- How to verify it Existing tests should pass.
- Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: --> N/A
- A picture of a cute animal (not mandatory)
pr created time in 6 days
issue commentdocker/compose
Allow colors for container log messages to be specified in docker-compose.yml
Hi @Greek64,
We see the Compose format as a way of modelling container based applications and use it not only for docker-compose but also for Swarm and Kubernetes (with docker stack deploy). As such, I don't think that a service level log colour in the format makes sense as other orchestrators wouldn't use it.
We could consider a docker-compose specific label or something though. What use case is not solved by the colours being deterministic as they are with #6996?
comment created time in 7 days
push eventdocker/app
commit sha fe99fdacbb35ac5996e5226488052447dc01da73
Add --digests flag to image ls Added the --digests flag to image ls. This makes the image ls command print the image digests (if present) along with the app name and image. Signed-off-by: Nick Adcock <[email protected]>
commit sha fd651ed687a579045773fd05c24843394c1eb02f
remove temp dirs from image list tests Removed the temporary directories from being created in the app image ls tests as they are not used. Signed-off-by: Nick Adcock <[email protected]>
commit sha 24ced43b0d04970183b3ad7f7a9eea606eedc399
split image ls output to repo and tag Split the app image ls output up so that the REPOSITORY and TAG are separate columns, the same as the standard docker image ls. The APP IMAGE ID column is now a default column, same as the ID column in docker image ls. As this ID is randomly generated the e2e tests have been updated to use regexes when asserting the command output. Signed-off-by: Nick Adcock <[email protected]>
commit sha 8acd0a838a7b0632f120fcfbb6443f897de62ad7
Merge pull request #709 from zappy-shu/add-digest-flag-to-ls Rework docker app image ls and add the --digests flag
push time in 7 days
PR merged docker/app
- WIP Need to get an e2e or unit test that covers the digest.
- What I did
Reworked the image ls output to better match the docker image ls command. The output now has separate columns for REPOSITORY and TAG as well as an APP IMAGE ID column.
Added the --digests flag to image ls. This makes the image ls command print the image digests (if present) along with the other columns.
- How I did it
Made the columns list for image ls dynamic depending on what options are sent.
The digest is retrieved by attempted to cast to a Digested type (i.e. the image has been stored by digest rather than by tag or ID). If not then just prints <none>.
- How to verify it
Pull an image from hub by digest and then run app image ls --digest. The image in question should show the digest prefixed by the algorithm type. E.g.
REPOSITORY TAG DIGEST APP IMAGE ID APP NAME
<repository> <none> sha256@<digest> <id> <name>
- Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->
Added the --digests flag to the app image ls command. When present, the command will print the image's digest or <none> if the image does not have a digest, along with other image details.
- A picture of a cute animal (not mandatory but encouraged)

pr closed time in 7 days
pull request commentdocker/app
Rework docker app image ls and add the --digests flag
Do we want to update the app tabwriter so the spacing match the CLI?
Yes. That will make the feel the same.
Probably best do the others as part of a separate PR though
👍 Please just create a ticket to track
comment created time in 7 days
pull request commentdocker/app
Rework docker app image ls and add the --digests flag
Is the spacing between the columns the same as for container images? From the following, It looks to me like app has less spaces.
➜ app git:(add-digest-flag-to-ls) docker app image ls
REPOSITORY TAG APP IMAGE ID APP NAME
ccrone/cnab-demo yves 6cca8f6428a9 coins
ccrone/cnab-demo yves 6cca8f6428a9 coins
➜ app git:(add-digest-flag-to-ls) docker images
REPOSITORY TAG IMAGE ID CREATED SIZE
docker/cnab-app-base v0.9.0-zeta1-66-g24ced43b0d f5d2426e6a7f 2 minutes ago 47.7MB
ccrone/cnab-demo <none> 51a16e185d83 59 minutes ago 47.2MB
ccrone/cnab-demo <none> 7cc9617f5c36 About an hour ago 218MB
ccrone/cnab-demo <none> de25a81a5a0b 2 weeks ago 98.2MB
comment created time in 7 days
Pull request review commentdocker/app
+FROM scratch++COPY . .
nit: \n
comment created time in 9 days
Pull request review commentdocker/app
+services:+ web:+ port: 1234
nit: \n
comment created time in 9 days
push eventchris-crone/cli
commit sha e4d3c360b24f36568180211045fa8b5675104452
Bump Compose on Kubernetes Signed-off-by: Christopher Crone <[email protected]>
push time in 13 days
push eventchris-crone/compose-on-kubernetes
commit sha 5f676d04e487f914dfa666c6ab887dda44dbe30b
WIP: Disable bind mount test Signed-off-by: Christopher Crone <[email protected]>
push time in 14 days
Pull request review commentdocker/compose-on-kubernetes
Better handle unstable releases
aliases: - attach_workspace: at: /root - setup_remote_docker:- version: 18.06.0-ce+ version: 18.09.3
nit: why not 19.03?
comment created time in 14 days
pull request commentdocker/compose
Copy over compose format 3.8 from docker/cli
I would argue that fields and changes made in docker/cli should override those in docker/compose for 3.x formats.
I would agree on this. We have an internal ticket tracking this, we just haven't prioritised it yet. I'll chat to our PM and see when we can fit it in.
comment created time in 14 days
Pull request review commentdocker/app
Fix handling short id in docker app commands
func runTag(bundleStore store.BundleStore, srcAppImage, destAppImage string) err } func readBundle(name string, bundleStore store.BundleStore) (*bundle.Bundle, error) {- cnabRef, err := store.StringToRef(name)+ cnabRef, err := bundleStore.LookUp(name) if err != nil {- return nil, err+ switch err.(type) {+ case *store.UnknownReferenceError:+ return nil, fmt.Errorf("could not tag '%s': no such App image", name)
Same question for line 56 and 63
comment created time in 14 days
Pull request review commentdocker/app
Fix handling short id in docker app commands
func ResolveBundle(dockerCli command.Cli, bundleStore appstore.BundleStore, name // GetBundle searches for the bundle locally and tries to pull it if not found func GetBundle(dockerCli command.Cli, bundleStore appstore.BundleStore, name string) (*bundle.Bundle, reference.Reference, error) {- ref, err := store.StringToRef(name)+ bndl, ref, err := getBundleFromStore(bundleStore, name) if err != nil {+ named, err := store.StringToNamedRef(name)+ if err != nil {+ return nil, nil, err+ }+ fmt.Fprintf(dockerCli.Err(), "Unable to find App image %q locally\n", reference.FamiliarString(named))+ fmt.Fprintf(dockerCli.Out(), "Pulling from registry...\n")+ bndl, err = PullBundle(dockerCli, bundleStore, named)+ if err != nil {+ return nil, nil, err+ }+ ref = named+ }+ return bndl, ref, nil+}++func getBundleFromStore(bundleStore appstore.BundleStore, name string) (*bundle.Bundle, reference.Reference, error) {+ ref, err := bundleStore.LookUp(name)+ if err != nil {+ logrus.Debugf(`Unable to find reference %q in the bundle store`, name)
nit: don't need back quotes here.
comment created time in 14 days
Pull request review commentdocker/app
Fix handling short id in docker app commands
func runTag(bundleStore store.BundleStore, srcAppImage, destAppImage string) err } func readBundle(name string, bundleStore store.BundleStore) (*bundle.Bundle, error) {- cnabRef, err := store.StringToRef(name)+ cnabRef, err := bundleStore.LookUp(name) if err != nil {- return nil, err+ switch err.(type) {+ case *store.UnknownReferenceError:+ return nil, fmt.Errorf("could not tag '%s': no such App image", name)
nit: is there a reason we use single quotes here but double quote elsewhere?
comment created time in 14 days
Pull request review commentdocker/app
Fix handling short id in docker app commands
func runTag(bundleStore store.BundleStore, srcAppImage, destAppImage string) err } func readBundle(name string, bundleStore store.BundleStore) (*bundle.Bundle, error) {- cnabRef, err := store.StringToRef(name)+ cnabRef, err := bundleStore.LookUp(name) if err != nil {- return nil, err+ switch err.(type) {+ case *store.UnknownReferenceError:+ return nil, fmt.Errorf("could not tag '%s': no such App image", name)
Looking further down, it appears this was the case from before
comment created time in 14 days
Pull request review commentdocker/app
Fix handling short id in docker app commands
func ResolveBundle(dockerCli command.Cli, bundleStore appstore.BundleStore, name // GetBundle searches for the bundle locally and tries to pull it if not found func GetBundle(dockerCli command.Cli, bundleStore appstore.BundleStore, name string) (*bundle.Bundle, reference.Reference, error) {- ref, err := store.StringToRef(name)+ bndl, ref, err := getBundleFromStore(bundleStore, name) if err != nil {+ named, err := store.StringToNamedRef(name)+ if err != nil {+ return nil, nil, err+ }+ fmt.Fprintf(dockerCli.Err(), "Unable to find App image %q locally\n", reference.FamiliarString(named))+ fmt.Fprintf(dockerCli.Out(), "Pulling from registry...\n")+ bndl, err = PullBundle(dockerCli, bundleStore, named)+ if err != nil {+ return nil, nil, err+ }+ ref = named+ }+ return bndl, ref, nil+}++func getBundleFromStore(bundleStore appstore.BundleStore, name string) (*bundle.Bundle, reference.Reference, error) {+ ref, err := bundleStore.LookUp(name)+ if err != nil {+ logrus.Debugf(`Unable to find reference %q in the bundle store`, name) return nil, nil, err } bndl, err := bundleStore.Read(ref) if err != nil {- fmt.Fprintf(dockerCli.Err(), "Unable to find App image %q locally\n", reference.FamiliarString(ref))-- fmt.Fprintf(dockerCli.Out(), "Pulling from registry...\n")- if named, ok := ref.(reference.Named); ok {- bndl, err = PullBundle(dockerCli, bundleStore, named)- if err != nil {- return nil, nil, err- }- }+ logrus.Debugf(`Unable to read bundle %q from store`, reference.FamiliarString(ref))
nit: don't need back quotes here.
comment created time in 14 days
pull request commentdocker/app
Add service image build example
I still need to add a readme for this so will leave it in draft until that's done.
comment created time in 14 days
push eventchris-crone/app
commit sha 6bc0a3fb12e05f8662e2d2819993163ec3d49c2b
Add service image build example Signed-off-by: Christopher Crone <[email protected]>
push time in 14 days
PR opened docker/app
- What I did Added an example of a Docker App that has service images that can be built.
- Description for the changelog N/A
pr created time in 14 days
pull request commentdocker/compose
This PR is included in the 1.25.0-rc3 release that you can find here. It will also be part of the next Docker Desktop edge release.
comment created time in 16 days
push eventchris-crone/app
commit sha 9c1bd62523998d516ba9d5e38058ca22b7f477c6
CI: Only clean images if there are any Signed-off-by: Christopher Crone <[email protected]>
push time in 16 days
push eventchris-crone/app
commit sha d00102713017ed0949b8d382900b5497d558adc8
CI: Only clean images if there are any Signed-off-by: Christopher Crone <[email protected]>
push time in 16 days
push eventchris-crone/app
commit sha 03f591935a482b49fbb46dc24c5cf63a7801acc6
Use docker build output for binaries Signed-off-by: Christopher Crone <[email protected]>
push time in 19 days
push eventchris-crone/app
commit sha b07e744df9a428c4bb8e390086222d6aa2070cfc
[b9bd12a5] Fix object naming in command help and update examples, e2E tests and file linting. Remove bracket changes in command usage message. Take into account feedback from Silvin, Chris and Nicolas. Signed-off-by: Caroline Briaud <[email protected]>
commit sha 6e215e76fb627208ad5e198c97838c3fa96e8bd5
Merge pull request #704 from carolinebriaud/fix-usages-examples Consistent object naming in usage command and examples
commit sha 2ec23c8874b7d2e339184cfba3c1f876015eb4f5
Use app naming rules in error and progress messages Signed-off-by: Caroline Briaud <[email protected]>
commit sha d20c612cf55c9e72d30da2d843e7b4a425aa5c38
Fix e2e Signed-off-by: Caroline Briaud <[email protected]>
commit sha 0abe6b4dbf581616e955ee40e014ffdbaaffd327
Fix another e2e Signed-off-by: Caroline Briaud <[email protected]>
commit sha 0ffdc1875e85fea06a3e969243e4f535743f5952
Merge pull request #713 from carolinebriaud/fix-usages-examples Use app naming rules in error and progress messages
commit sha 162131de180999b1b3a4799922c173577a5fc67f
Activate BuildKit on CI Signed-off-by: Christopher Crone <[email protected]>
commit sha 0d08d65d197bca24f6c2cc7628b8b6971457d5f0
Merge pull request #714 from chris-crone/buildkit-ci Activate BuildKit on CI
commit sha d82f79998a6c57649b2ac5d2775f2c5599e5e752
Use docker build output for binaries Signed-off-by: Christopher Crone <[email protected]>
push time in 19 days
PR merged docker/app
- What I did Activated BuildKit for the CI.
- How I did it
Set DOCKER_BUILDKIT=1 in the Jenkinsfiles.
- How to verify it
- Description for the changelog N/A
- A picture of a cute animal (not mandatory but encouraged)
pr closed time in 19 days
push eventdocker/app
commit sha 162131de180999b1b3a4799922c173577a5fc67f
Activate BuildKit on CI Signed-off-by: Christopher Crone <[email protected]>
commit sha 0d08d65d197bca24f6c2cc7628b8b6971457d5f0
Merge pull request #714 from chris-crone/buildkit-ci Activate BuildKit on CI
push time in 19 days
PR opened docker/app
- What I did Activated BuildKit for the CI.
- How I did it
Set DOCKER_BUILDKIT=1 in the Jenkinsfiles.
- How to verify it
- Description for the changelog N/A
- A picture of a cute animal (not mandatory but encouraged)
pr created time in 19 days
pull request commentdocker/app
Use docker build output for binaries
I think this is because we don't use BuildKit on the CI
comment created time in 19 days
push eventchris-crone/app
commit sha 0de7446c06ca7525af85b9d76debd24d1557b7bb
Use docker build output for binaries Signed-off-by: Christopher Crone <[email protected]>
push time in 19 days
Pull request review commentdocker/cli
CircleCI: add some info steps, and re-group jobs
jobs: reusable: true exclusive: false - run:+ name: "Docker version" command: docker version - run:- name: "Lint"+ name: "Docker info"+ command: docker info+ - run:+ name: "Lint - build image" command: | docker build --progress=plain -f dockerfiles/Dockerfile.lint --tag cli-linter:$CIRCLE_BUILD_NUM .+ - run:
nit: which is quicker lint or spell check? I'd put the quicker one first so that we can fail faster
comment created time in 19 days
PR opened docker/app
- What I did
Leveraged docker build --output to get artefacts from images. This means that we can avoid doing docker build && docker container create && docker cp && docker rm.
- How I did it
- How to verify it Existing tests should all pass.
- Description for the changelog N/A
- A picture of a cute animal (not mandatory but encouraged)
pr created time in 19 days
Pull request review commentdocker/app
Consistent object naming in usage command and examples
func muteDockerCli(dockerCli command.Cli) func() { func inspectCmd(dockerCli command.Cli) *cobra.Command { var opts inspectOptions cmd := &cobra.Command{- Use: "inspect [APP_NAME] [OPTIONS]",- Short: "Shows metadata, parameters and a summary of the Compose file for a given application",- Example: `$ docker app inspect my-installed-app-$docker app inspect my-app:1.0.0`,+ Use: "inspect [OPTIONS] APP_IMAGE",+ Short: "Display detailed information on an App image",
s/on/about/
comment created time in 20 days
push eventthaJeztah/app
commit sha 472296922536b5817e7c764b37569d90af41950e
Add --tag flag to the bundle command to persist the bundle in the local bundle store. Installation or any other command will be able to use this reference. Invocation image is using the same name, appending "-invoc" to the tag. Signed-off-by: Silvin Lubecki <[email protected]>
commit sha 5a5671062f878c5bf49c159668c2bacff82820ad
inspect: sort services, networks, volumes and secrets by name Networks, Volumes and Secrets are maps so iteration is randomized by the runtime. Although Services are an slice here they have, at some point, be laundered through a map and so also need sorting. Test by adding a second instance of each to the "full" test case and by always inspecting twice (in case the first one got lucky). Fixes: #525 Signed-off-by: Ian Campbell <[email protected]>
commit sha e8e4bb3bd7a5153eeb404aa2b11b6895c17112a3
Add the possibility to configure the base invocation image in the CLI config file. Print invocation base image version in the version command output. The docker app version command has a new --base-invocation-image flag which prints only the name the of image. Useful in order to pull or save the image. Signed-off-by: Jean-Christophe Sirot <[email protected]>
commit sha 8aacca23ef721ac2eabf24c8e5841bd9dd47a496
Merge pull request #521 from jcsirot/offline-base-cnab Provide support for using docker app offline
commit sha 257a6538b895b3ed67638b5110343bb69a286a45
e2e: Close the config file after writing it. Signed-off-by: Ian Campbell <[email protected]>
commit sha 9ece4d778f8926efd074087b15d44b772f14f91d
Allow the user to specify individual credentials on the command line e.g. docker app install --credential name=somevalue bundle.json Credentials added with `--credential` always come after those added with `--credential-set` (irrespective of the order on the command line). A credential specified with `--credential` cannot override any previous credential, including those specified in a credential set. The test bnudle used is based on https://github.com/deislabs/example-bundles/blob/0e8af9a2f1270bd72045a515637a432e74743d5d/example-credentials/bundle.json But with `cnab/example-credentials:latest` → a digested ref (with the digest I pulled today) Signed-off-by: Ian Campbell <[email protected]>
commit sha 368989a03ff6c12e08bd821e526a02e1e8cf8e49
Merge pull request #520 from silvin-lubecki/bundle-with-tag Bundle command now saves the bundle to the bundle store
commit sha 96d4d9bec487e747ef3a55ba8b30fba4c9039d27
metadata: Add helper to extract docker-app metadata from a bundle. Signed-off-by: Ian Campbell <[email protected]>
commit sha 8d7750b30d3584a5a46665ddce778eebd31d3126
push: wrap some errors so the error message has more context. Turns: denied: requested access to the resource is denied into: pushing to "docker.io/library/hello-world:0.1.0-invoc": denied: requested access to the resource is denied Signed-off-by: Ian Campbell <[email protected]>
commit sha 11955fe9da0dde16518a436d7204e9ed38d4383c
push: support pushing directly from a bundle file. This is done by providing a new `--bundle` option. Signed-off-by: Ian Campbell <[email protected]>
commit sha eca2252364a6bf59c9979e1a842847b14516b59a
push: autodetect when asked to push a bundle ... instead of requiring a separate option, this follows the pattern of `install` et al. Signed-off-by: Ian Campbell <[email protected]>
commit sha 594ce0efb50b73275c4f0a0a6a265fbb75a57b6c
e2e: Use subtests in render test. Separate the "file" and "stdio" case since they both error with "rendering mismatch". Signed-off-by: Ian Campbell <[email protected]>
commit sha ced138ba830f975b8d733b3b005e8fc9492a5d36
e2e: Add a test for pushing from a bundle and from a ref The use of `!strings.Contains()` in one assertion, rather than `!cmp.Contains()` is due to https://github.com/gotestyourself/gotest.tools/issues/147 Signed-off-by: Ian Campbell <[email protected]>
commit sha b9539583525ec06cc086e7aca0f2a34e7c1f245f
Merge pull request #529 from ijc/push-from-bundle Push directly from bundle
commit sha f2fbcae871cd80ba2389fef50879b961c0a2cee5
Merge pull request #531 from ijc/individual-credentials Allow the user to specify individual credentials on the command line
commit sha b4a8a4a6f97c0289339d9a375b19641f2d1e7723
Bump docker/cli Bump docker/docker Fix last changes on context stores made on docker/cli (the main store interface has been split to multiple smaller interfaces). Signed-off-by: Silvin Lubecki <[email protected]>
commit sha bbeedd73b103ac278537df5dd054b785de5f7eed
Bumping alpine 3.9.3 -> 3.9.4 Bumping golang/golang-cross 1.12.4 -> 1.12.5 Signed-off-by: Silvin Lubecki <[email protected]>
commit sha f5f38732b0044b06064c5330b250f78d5249bd6a
Merge pull request #538 from silvin-lubecki/bump Bumping vendoring and image dependencies
commit sha cbdc83942eb28a4f16a7bee4029823109104067b
Mark the app plugin as an experimental plugin. Only a cli with experimental flag enabled will list docker app as a plugin. Bump docker/cli to the last 19.03 Signed-off-by: Silvin Lubecki <[email protected]>
commit sha 73d867442d404f3e545e67b80b65545421d90e65
Merge pull request #537 from silvin-lubecki/experimental-plugin Mark docker app plugin as experimental
push time in 20 days
Pull request review commentdocker/app
func buildImageUsingBuildx(app *types.App, contextPath string, opt buildOptions, Driver: d, }, }- pw := progress.NewPrinter(ctx, os.Stderr, opt.progress)++ var out *os.File+ if opt.quiet {+ if out, err = os.Create(os.DevNull); err != nil {+ return nil, err+ }+ } else {+ out = os.NewFile(dockerCli.Out().FD(), "/dev/stdout")
Does /dev/stdout work on Windows?
comment created time in 21 days
Pull request review commentdocker/app
Consistent object naming in usage command and examples
func runCmd(dockerCli command.Cli) *cobra.Command { } opts.parametersOptions.addFlags(cmd.Flags()) opts.credentialOptions.addFlags(cmd.Flags())- cmd.Flags().StringVar(&opts.orchestrator, "orchestrator", "", "Orchestrator to install on (swarm, kubernetes)")- cmd.Flags().StringVar(&opts.kubeNamespace, "namespace", "default", "Kubernetes namespace to install into")- cmd.Flags().StringVar(&opts.stackName, "name", "", "Assign a name to the installation")+ cmd.Flags().StringVar(&opts.orchestrator, "orchestrator", "", "Orchestrator to run on (swarm, kubernetes)")+ cmd.Flags().StringVar(&opts.kubeNamespace, "namespace", "default", "Kubernetes namespace where to run the App")
s/where/in which/
comment created time in 21 days
Pull request review commentdocker/app
Consistent object naming in usage command and examples
type renderOptions struct { func renderCmd(dockerCli command.Cli) *cobra.Command { var opts renderOptions cmd := &cobra.Command{- Use: "render [APP_NAME] [--set KEY=VALUE ...] [--parameters-file PARAMETERS-FILE ...] [OPTIONS]",- Short: "Render the Compose file for an Application Package",- Example: `$ docker app render myapp.dockerapp --set key=value`,+ Use: "render [APP_DEFINITION] [OPTIONS]",+ Short: "Render the Compose file for an App Definition",
@silvin-lubecki does render work from definitions as well as images?
comment created time in 21 days
Pull request review commentdocker/app
Consistent object naming in usage command and examples
import ( func rmCmd() *cobra.Command { return &cobra.Command{- Short: "Remove an application image",+ Short: "Remove an App Image", Use: "rm [APP_IMAGE] [APP_IMAGE...]", Aliases: []string{"remove"}, Args: cli.RequiresMinArgs(1), Example: `$ docker app image rm myapp $ docker app image rm myapp:1.0.0-$ docker app image rm docker.io/library/myapp@sha256:beef...`,+$ docker app image rm myrepo/myapp@sha256:beef...`,
nit: we could make the SHA prefix something like c0de instead of beef given that we're making developer tools
comment created time in 21 days
Pull request review commentdocker/app
Consistent object naming in usage command and examples
Usage: docker app [OPTIONS] COMMAND -A tool to build and manage Docker Applications.+A tool to build, share and run a Docker App Options: --version Print version information Management Commands:- image Manage application images+ image Manage App Images Commands:- build Build service images for the application- init Initialize Docker Application definition- ls List the installations and their last known installation result- pull Pull an application package from a registry- push Push an application package to a registry- rm Remove an application- run Run an application- update Update a running application- validate Checks the rendered application is syntactically correct+ build Build an App Image from a .dockerapp App Definition+ init Initialize an App Definition+ ls List Running Apps
Do we want to capitalise Running and Image?
Note that image is not capitalised in the CLI:
$ docker image help
Usage: docker image COMMAND
Manage images
comment created time in 21 days
PR closed docker/app
Signed-off-by: Christopher Crone [email protected]
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/docker/app/blob/master/CONTRIBUTING.md
** Make sure all your commits include a signature generated with git commit -s **
For additional information on our contributing process, read our contributing guide https://docs.docker.com/opensource/code/
If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx"
Please provide the following information: -->
- What I did As the YAML library bump is now in the 19.03 branch for the CLI, I've moved our vendoring back to use that.
- How I did it
- Update
Gopkg.toml - Run
make vendor
- How to verify it
- Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: -->
- A picture of a cute animal (not mandatory but encouraged)
pr closed time in 21 days
pull request commentdocker/app
[WIP] Move docker/cli vendoring back to 19.03 branch
We can bump this later
comment created time in 21 days
push eventchris-crone/compose-on-kubernetes
commit sha 3ffb3157da140c547d311da04d436a4862b43ec1
WIP: Comment out test Signed-off-by: Christopher Crone <[email protected]>
push time in a month
push eventchris-crone/compose-on-kubernetes
push time in a month
pull request commentdocker/compose
Update engine to 19.03.3, and Alpine 3.10.2
Although the Dockerfile defaults to Python 3.7.4, the build-debian stage installs python2.7-dev - is that correct? https://github.com/docker/compose/blob/79f29dd/Dockerfile#L43
Not sure about this, @ulyssessouza can answer when he's back
The Dockerfile still has a FIXME, and pins to (older?) versions of virtualenv and tox. What's the easiest way to find out if those issues have been resolved? Will there be an error when updating to later versions, or does the breaking stuff happen "silently"? https://github.com/docker/compose/blob/79f29dd/Dockerfile#L51-L53
The issue was that it broke the macOS build in a way that was not obvious. I haven't tried newer versions recently to see if this is fixed. virtualenv and tox are used for testing and build purposes so we're not shipping them.
comment created time in a month
push eventchris-crone/compose-on-kubernetes
commit sha b66d6c2a38cf800ca818a5778519401067b174dc
Do not run tests in parallel on CI Signed-off-by: Christopher Crone <[email protected]>
push time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 1c80fc7ebd99e1fb47af9e037eabd19f4238b673
Bump etcd used in tests to v3.3.15 Signed-off-by: Christopher Crone <[email protected]>
push time in a month
PR merged docker/app
- What I did
Renamed the upgrade command to update
- How to verify it
Run docker app --help
- A picture of a cute animal (not mandatory but encouraged)

pr closed time in a month
push eventdocker/app
commit sha 32da7d22e7d80a9f0196d7fc6428c6306089bfd8
Rename upgrade to update Signed-off-by: Djordje Lukic <[email protected]>
commit sha 8f1a5943a49e6f291f40408a74ab927962ee4d75
Merge pull request #694 from rumpl/ux-upgrade-update Rename upgrade to update
push time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 3ecadbd009875e646290bafa9a8002ae3f07bfbb
Fix controller health check for tests Signed-off-by: Christopher Crone <[email protected]>
push time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 5a6c5701454305ef700851587b1dbacc8721aea9
Fix controller health check for tests Signed-off-by: Christopher Crone <[email protected]>
push time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 4124ea2f90521706af5a85688868db3613523d8c
Fix controller health check Signed-off-by: Christopher Crone <[email protected]>
push time in a month
PR opened docker/cli
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/docker/cli/blob/master/CONTRIBUTING.md
** Make sure all your commits include a signature generated with git commit -s **
For additional information on our contributing process, read our contributing guide https://docs.docker.com/opensource/code/
If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx"
Please provide the following information: -->
- What I did Bumped Compose on Kubernetes
- How I did it
- Updated
vendor.conf - Added k8s.io dependencies
vndr
- How to verify it
- Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: -->
- A picture of a cute animal (not mandatory but encouraged)
pr created time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 568febf14d885d412aadfa42521c651c2e7870f2
Do not advertise protobuf support This fixes https://github.com/docker/compose-on-kubernetes/issues/150. As the `Stack` struct does not support being serialized to protobuf, we must not advertise that we support protobuf. Signed-off-by: Christopher Crone <[email protected]>
push time in a month
Pull request review commentdocker/compose
avoid using Stty, rely on Posix compliant os.get_terminal_size
+backports.shutil_get_terminal_size==1.0.0I think you'll need to add this to setup.py too so that it's included when people do a pip install
comment created time in a month
issue commentdocker/compose-on-kubernetes
k8s 1.16: serialization issue, namespace deletion hangs
It appears that while all other requests to the Compose API Server use JSON, delete collection (which is called as part of delete namespace) always uses protobuf – even if we register the API Server as only having JSON and YAML support. I think this is a Kubernetes 1.16 bug and we are looking for a work around.
comment created time in a month
Pull request review commentdocker/compose-on-kubernetes
Bump Kubernetes vendoring to 1.16.1
func setConfigDefaults(config *rest.Config) error { gv := v1alpha3.SchemeGroupVersion config.GroupVersion = &gv config.APIPath = "/apis"- config.NegotiatedSerializer = serializer.DirectCodecFactory{CodecFactory: scheme.Codecs}+ config.NegotiatedSerializer = serializer.WithoutConversionCodecFactory{CodecFactory: scheme.Codecs}
Note to reviewer: This change was based on this: https://github.com/kubernetes/kubernetes/pull/76270
comment created time in a month
Pull request review commentdocker/compose-on-kubernetes
Bump Kubernetes vendoring to 1.16.1
func TestConvertTable(t *testing.T) { { name: "not a stack", object: &metav1beta1.Table{},- expectedError: "unexpected object type *v1beta1.Table",+ expectedError: "unexpected object type *v1.Table",
Note to reviewer that this change was needed
comment created time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 698549f303255f89578640a4fab81c9a590f411d
Bump vendoring Primary action was bumping the Kubernetes vendoring, other operations were to make it work. * Kubernetes to 1.16.1 * Removed `bitbucket.org/ww/goautoneg` as it is now included in Prometheus * Changed `serializer.DirectCodecFactory` to `serializer.WithoutConversionCodecFactory` in `api/client/clientset/typed/compose/*/compose_client.go` * Moved grpc to v1.23.1 * Ran `dep ensure -update sigs.k8s.io/structured-merge-diff` * Ran `dep ensure -update k8s.io/utils` Signed-off-by: Christopher Crone <[email protected]>
push time in a month
PR opened docker/compose-on-kubernetes
Additional information:
- Removed
bitbucket.org/ww/goautonegas it is now included in Prometheus - Changed
serializer.DirectCodecFactorytoserializer.WithoutConversionCodecFactoryinapi/client/clientset/typed/compose/*/compose_client.go - Moved grpc to v1.23.1
- Ran
dep ensure -update sigs.k8s.io/structured-merge-diff - Ran
dep ensure -update k8s.io/utils
pr created time in a month
push eventchris-crone/compose-on-kubernetes
commit sha 1bb60ce1fe49f5f7a29aba84d3263c2d0a9b0ce9
Bump vendoring Primary action was bumping the Kubernetes vendoring, other operations were to make it work. * Kubernetes to 1.16.1 * Removed `bitbucket.org/ww/goautoneg` as it is now included in Prometheus * Changed `serializer.DirectCodecFactory` to `serializer.WithoutConversionCodecFactory` in `api/client/clientset/typed/compose/*/compose_client.go` * Moved grpc to v1.23.1 * Ran `dep ensure -update sigs.k8s.io/structured-merge-diff` * Ran `dep ensure -update k8s.io/utils` Signed-off-by: Christopher Crone <[email protected]>
push time in a month
create barnchchris-crone/compose-on-kubernetes
created branch time in a month
Pull request review commentdocker/app
Add warnings for env_file entries not copied
func checkComposeFileVersion(compose map[string]interface{}) error { return schema.Validate(compose, fmt.Sprintf("%v", version)) } -func initFromComposeFile(name string, composeFile string) error {- logrus.Debugf("Initializing from compose file %s", composeFile)-- dirName := internal.DirNameFromAppName(name)-- composeRaw, err := ioutil.ReadFile(composeFile)- if err != nil {- return errors.Wrap(err, "failed to read compose file")+func getEnvFiles(svcName string, envFileEntry interface{}) ([]string, error) {+ var envFiles []string+ switch envFileEntry.(type) {+ case string:+ envFiles = append(envFiles, envFileEntry.(string))+ case []interface{}:+ for _, env := range envFileEntry.([]interface{}) {+ envFiles = append(envFiles, env.(string))+ }+ default:+ return nil, fmt.Errorf("unknown entries in 'env_file' for service %s -> %v",+ svcName, envFileEntry) }- cfgMap, err := composeloader.ParseYAML(composeRaw)- if err != nil {- return errors.Wrap(err, "failed to parse compose file")+ return envFiles, nil+}++func checkEnvFiles(cfgMap map[string]interface{}) error {+ services := cfgMap["services"]+ servicesMap, ok := services.(map[string]interface{})+ if !ok {+ return fmt.Errorf("could not process the yaml file") }- if err := checkComposeFileVersion(cfgMap); err != nil {- return err+ for svcName, svc := range servicesMap {+ svcContent, ok := svc.(map[string]interface{})+ if !ok {+ return fmt.Errorf("could not process the yaml file")+ }+ envFileEntry, ok := svcContent["env_file"]+ if !ok {+ continue+ }+ envFiles, err := getEnvFiles(svcName, envFileEntry)+ if err != nil {+ return errors.Wrap(err, "could not process the yaml file")+ }+ for _, envFilePath := range envFiles {+ fmt.Printf("\"env_file: %s\" entry of service %q will NOT be copied automatically! "+
I would say:
"%s.env_file %q will not be copied into %s.dockerapp. Please copy it manually and udpate the path accordingly in the compose file.", svcName, appName, envFilePath
comment created time in a month
Pull request review commentdocker/app
Add warnings for env_file entries not copied
func checkComposeFileVersion(compose map[string]interface{}) error { return schema.Validate(compose, fmt.Sprintf("%v", version)) } -func initFromComposeFile(name string, composeFile string) error {- logrus.Debugf("Initializing from compose file %s", composeFile)-- dirName := internal.DirNameFromAppName(name)-- composeRaw, err := ioutil.ReadFile(composeFile)- if err != nil {- return errors.Wrap(err, "failed to read compose file")+func getEnvFiles(svcName string, envFileEntry interface{}) ([]string, error) {+ var envFiles []string+ switch envFileEntry.(type) {+ case string:+ envFiles = append(envFiles, envFileEntry.(string))+ case []interface{}:+ for _, env := range envFileEntry.([]interface{}) {+ envFiles = append(envFiles, env.(string))+ }+ default:+ return nil, fmt.Errorf("unknown entries in 'env_file' for service %s -> %v",+ svcName, envFileEntry) }- cfgMap, err := composeloader.ParseYAML(composeRaw)- if err != nil {- return errors.Wrap(err, "failed to parse compose file")+ return envFiles, nil+}++func checkEnvFiles(cfgMap map[string]interface{}) error {+ services := cfgMap["services"]+ servicesMap, ok := services.(map[string]interface{})+ if !ok {+ return fmt.Errorf("could not process the yaml file") }- if err := checkComposeFileVersion(cfgMap); err != nil {- return err+ for svcName, svc := range servicesMap {+ svcContent, ok := svc.(map[string]interface{})+ if !ok {+ return fmt.Errorf("could not process the yaml file")+ }+ envFileEntry, ok := svcContent["env_file"]+ if !ok {+ continue+ }+ envFiles, err := getEnvFiles(svcName, envFileEntry)+ if err != nil {+ return errors.Wrap(err, "could not process the yaml file")
s/compose/Compose/
comment created time in a month
Pull request review commentdocker/app
Add warnings for env_file entries not copied
func checkComposeFileVersion(compose map[string]interface{}) error { return schema.Validate(compose, fmt.Sprintf("%v", version)) } -func initFromComposeFile(name string, composeFile string) error {- logrus.Debugf("Initializing from compose file %s", composeFile)-- dirName := internal.DirNameFromAppName(name)-- composeRaw, err := ioutil.ReadFile(composeFile)- if err != nil {- return errors.Wrap(err, "failed to read compose file")+func getEnvFiles(svcName string, envFileEntry interface{}) ([]string, error) {+ var envFiles []string+ switch envFileEntry.(type) {+ case string:+ envFiles = append(envFiles, envFileEntry.(string))+ case []interface{}:+ for _, env := range envFileEntry.([]interface{}) {+ envFiles = append(envFiles, env.(string))+ }+ default:+ return nil, fmt.Errorf("unknown entries in 'env_file' for service %s -> %v",+ svcName, envFileEntry) }- cfgMap, err := composeloader.ParseYAML(composeRaw)- if err != nil {- return errors.Wrap(err, "failed to parse compose file")+ return envFiles, nil+}++func checkEnvFiles(cfgMap map[string]interface{}) error {+ services := cfgMap["services"]+ servicesMap, ok := services.(map[string]interface{})+ if !ok {+ return fmt.Errorf("could not process the yaml file")
s/compose/Compose/
comment created time in a month
push eventdocker/compose
commit sha 37be2ad9cd54e61de07a43c14ca768b2d07da851
Remove set -x to make this script less verbose Signed-off-by: Jean-Christophe Sirot <[email protected]>
commit sha 8ebb1a6f19dad6fd20a071301bb460ee5c60003f
Merge pull request #6949 from jcsirot/fix-pushbin-script-verbosity Remove set -x to make this script less verbose
push time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/xeipuuv/gojsonschema from 93e72a773fade158921402d6a24c819b48aba29d to 1.1.0. This release includes the previously tagged commit. <details> <summary>Release notes</summary>
Sourced from github.com/xeipuuv/gojsonschema's releases.
v1.1.0
f971f3cReplace big.Float by big.Ratcdd33d0Update documentation and add draft crossreference test7cd9691Add an extra test for schema detection and fix some bugse8d857cAdd support for custom metaschemas using $schema42c3b50Disable certain features if a specific draft is usedc813aa8Handle (exclusive)minimum/maximum with different draftsdaa7675Fully integrate draft version detection2f08306First work to implement draft differentiationac52e68Add indexes not unique array items to error detailsf3a9daeRemoved if statement in result.go- Additional commits viewable in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/evanphx/json-patch from 5858425f75500d40c52783dce87d085a483ce135 to 4.5.0. This release includes the previously tagged commit. <details> <summary>Release notes</summary>
Sourced from github.com/evanphx/json-patch's releases.
Improve Errors
This release improves the errors returned by using
github.com/pkg/errorsto allow for better error taxonomy detection. </details> <details> <summary>Commits</summary>
026c730Cleanup errors to help understand what failed. Fixes #75aad5688Merge pull request #79 from sbward/master72d00a9Revert govet fixes250db64Update error messages9d7a8dfExpose Operation and methods, fix lint errors- See full diff in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/onsi/ginkgo from e51bee6 to 5049dc5.
<details>
<summary>Commits</summary>
5049dc5speed up table entry generateIt() (#609)7bb3091Fix. Write errors to stderr instead of stdout (#610)974566cv1.10.12a4c0bdstack backtrace: fix skipping (#600)f90f37dv1.10.066915d6stack backtrace: fix alignment and skipping8f97b93fix typo in documentation7e8b50dv1.9.00ecbc58Fixed typos in commentsa7f8bfbgofmt code- Additional commits viewable in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/Masterminds/semver from 1.4.2 to 3.0.1. <details> <summary>Release notes</summary>
Sourced from github.com/Masterminds/semver's releases.
3.0.1
#125: Fixes issue with module path for v3
v3.0.0
This is a major release of the semver package which includes API changes. The Go API is compatible with ^1. The Go API was not changed because many people are using
go getwithout Go modules for their applications and API breaking changes cause errors which we have or would need to support.The changes in this release are the handling based on the data passed into the functions. These are described in the added and changed sections below.
Added
- StrictNewVersion function. This is similar to NewVersion but will return an error if the version passed in is not a strict semantic version. For example, 1.2.3 would pass but v1.2.3 or 1.2 would fail because they are not strictly speaking semantic versions. This function is faster, performs fewer operations, and uses fewer allocations than NewVersion.
- Fuzzing has been performed on NewVersion, StrictNewVersion, and NewConstraint. The Makefile contains the operations used. For more information on you can start on Wikipedia at https://en.wikipedia.org/wiki/Fuzzing
- Now using Go modules
Changed
- NewVersion has proper prerelease and metadata validation with error messages to signal an issue with either of them
- Handles space separated AND conditions in addition to ,
- ^ now operates using a similar set of rules to npm/js and Rust/Cargo. If the version is >=1 the ^ ranges works the same as v1. For major versions of 0 the rules have changed. The minor version is treated as the stable version unless a patch is specified and then it is equivalent to =. One difference from npm/js is that prereleases there are only to a specific version (e.g. 1.2.3). Prereleases here look over multiple versions and follow semantic version ordering rules. This pattern now follows along with the expected and requested handling of this packaged by numerous users.
Release 1.5.0
Added
Changed
- #82: Clarify wildcard meaning in range constraints and update tests for it (thanks @greysteil)
- #83: Clarify caret operator range for pre-1.0.0 dependencies (thanks @greysteil)
- #72: Adding docs comment pointing to vert for a cli
- #71: Update the docs on pre-release comparator handling
- #89: Test with new go versions (thanks @thedevsaddam) </tr></table> ... (truncated) </details> <details> <summary>Changelog</summary>
Sourced from github.com/Masterminds/semver's changelog.
3.0.1 (2019-09-13)
Fixed
- #125: Fixes issue with module path for v3
3.0.0 (2019-09-12)
This is a major release of the semver package which includes API changes. The Go API is compatible with ^1. The Go API was not changed because many people are using
go getwithout Go modules for their applications and API breaking changes cause errors which we have or would need to support.The changes in this release are the handling based on the data passed into the functions. These are described in the added and changed sections below.
Added
- StrictNewVersion function. This is similar to NewVersion but will return an error if the version passed in is not a strict semantic version. For example, 1.2.3 would pass but v1.2.3 or 1.2 would fail because they are not strictly speaking semantic versions. This function is faster, performs fewer operations, and uses fewer allocations than NewVersion.
- Fuzzing has been performed on NewVersion, StrictNewVersion, and NewConstraint. The Makefile contains the operations used. For more information on you can start on Wikipedia at https://en.wikipedia.org/wiki/Fuzzing
- Now using Go modules
Changed
- NewVersion has proper prerelease and metadata validation with error messages to signal an issue with either of them
- ^ now operates using a similar set of rules to npm/js and Rust/Cargo. If the version is >=1 the ^ ranges works the same as v1. For major versions of 0 the rules have changed. The minor version is treated as the stable version unless a patch is specified and then it is equivalent to =. One difference from npm/js is that prereleases there are only to a specific version (e.g. 1.2.3). Prereleases here look over multiple versions and follow semantic version ordering rules. This pattern now follows along with the expected and requested handling of this packaged by numerous users.
1.5.0 (2019-09-11)
Added
Changed
- #82: Clarify wildcard meaning in range constraints and update tests for it (thanks @greysteil) </tr></table> ... (truncated) </details> <details> <summary>Commits</summary>
fe7c210Merge pull request #126 from mattfarina/modfix7e17568Fixes issue with module path for v30337c9bMerge pull request #124 from mattfarina/v3-rel-notesbe888dfUpdating the docs for the 3.0.0 release4492473Merge pull request #121 from mattfarina/fuzzingb01f0c4Adding fuzzing for v33c3e3c7Merge pull request #123 from mattfarina/changelog-update3c4400c9Updating changelog for 1.5.0 release0873dd6Merge pull request #120 from mattfarina/fix-1196d097caHandling ^ correctly with additional tests- Additional commits viewable in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/stretchr/testify from 1.2.2 to 1.4.0. <details> <summary>Release notes</summary>
Sourced from github.com/stretchr/testify's releases.
v1.4.0
The 1.4.0 release includes new matchers and bug fixes. See the v.1.4.0 milestone for a complete list of closed issues associated with this release.
v1.3.0
Pull requests included in this release:
- stretchr/testify#678 - Fixed incorrect comment
- stretchr/testify#661 - Allow assert.Equal on string type alias without panicking on failure
- stretchr/testify#659 - Add Go1.11 to CI
- stretchr/testify#682 - update comment to match function name
- stretchr/testify#688 - Bump Go versions and use '.x' to always get latest minor versions
- stretchr/testify#648 - Update the LICENSE to match the 'standard' MIT license file
- stretchr/testify#699 - Support arbitrary objects as the sole element in msgAndArgs
- stretchr/testify#696 - Recover panic in suite
- stretchr/testify#655 - Add support for subtests (go 1.7)
- stretchr/testify#607 - Add go.mod and go.modverify
- stretchr/testify#670 - Fix typo ("PASS" -> "FAIL")
- stretchr/testify#712 - Preserve stack frame for mock parent method call
- stretchr/testify#707 - fix for comparing kinds
- stretchr/testify#697 - Update readme to deprecate http and include available packages </details> <details> <summary>Commits</summary>
221dbe5fix mutex1bb3d5aimprove find expected call555ebd3Fix panic for equality assertions for values of aliased string type0224ef2Add YAMLEq to assert that two YAML strings are equivalentd84e815Introduce Eventually assertion.ae5876dchange report wording, fix int stringification, fix doc34c6fa2assert, require: Regenerate21cb1c2add ptr comparison func 'Same' to assert pkg10a9f47Show stack trace on panic3f658bdadd comparison- Additional commits viewable in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/mitchellh/go-homedir from 3864e76 to af06845.
<details>
<summary>Commits</summary>
af06845Add a Reset functionae18d6bAdd go.mod and release v15804607Fix formattingd26f376Merge pull request #24 from mbrancato/master32e698aShift up the preference of the USERPROFILE variable- See full diff in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR closed docker/compose-on-kubernetes
Bumps github.com/onsi/gomega from 9be8410 to bdebf9e.
<details>
<summary>Commits</summary>
bdebf9ev1.7.0642e5baexport format property variables (#347)beea727minor fix in the documentation of ExpectWithOffset (#358)27d50eav1.6.0cc1a6cbRemove duplication in XML matcher tests7bf756aRemove unnecessary conversions (#357)2e3b965Fixed import order (#353)c98d3ebAdded missing error handling in test (#355)0001ed9Simplify code (#356)0d9100eSimplify code (#354)- Additional commits viewable in compare view </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.
You can always request more updates by clicking Bump now in your Dependabot dashboard.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language@dependabot badge mewill comment on this PR with code to add a "Dependabot enabled" badge to your readme
Additionally, you can set the following in your Dependabot dashboard:
- Update frequency (including time of day and day of week)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)
Finally, you can contact us by mentioning @dependabot.
</details>
pr closed time in a month
PR opened docker/compose-on-kubernetes
The kind versions that I had specified don't exist but are only part of the master/tag CI pipeline so I didn't catch it. This should fix the CI.
pr created time in a month
create barnchchris-crone/compose-on-kubernetes
created branch time in a month
push eventchris-crone/compose-on-kubernetes
commit sha fb230e0b2a060c677a71de266d705af48a45a45c
Move from apps/v1beta2 to apps/v1 Signed-off-by: Christopher Crone <[email protected]>
commit sha d1e8a5fb15138fe6f48c4427cbef6383bd409daf
Test Kubernetes 1.16 Signed-off-by: Christopher Crone <[email protected]>
push time in a month
push eventchris-crone/compose-on-kubernetes
commit sha a2368c5e052bed604b96b91d23cda3d33d5a4c66
vendor: Bump Kubernetes to 1.14.7 Signed-off-by: Christopher Crone <[email protected]>
commit sha 76adfbf56efb754980897cc564a47e421ffc0f5d
vendor: Bump etcd and gRPC Signed-off-by: Christopher Crone <[email protected]>
commit sha a81f27d4a851cc2ce9f9d3c2422b46b39bad9900
Merge pull request #146 from chris-crone/bump-vendor Bump Kubernetes and etcd
commit sha fc232faf4a7f6ef5fcae2a76608ca3f23c1e0a52
Use upstream yaml.v2 To mitigate against malicious YAML (as described here: https://github.com/kubernetes/kubernetes/issues/83253) we used a patched version of yaml.v2. There is now a fix upstream so we can leverage that. Signed-off-by: Christopher Crone <[email protected]>
commit sha 1eb05cfb035349f1c26f5ebb2cacd90c9a694718
Merge pull request #145 from chris-crone/upstream-yaml.v2 Use upstream yaml.v2
commit sha 3d63edb8c6a8e4786b6de8150fe4effc1b76240e
Move from apps/v1beta2 to apps/v1 Signed-off-by: Christopher Crone <[email protected]>
commit sha 4fcdd0d5957f9213b20c6317064b6431d1ea29de
Add Kubernetes 1.16 to e2e matrix Signed-off-by: Christopher Crone <[email protected]>
push time in a month
pull request commentdocker/docker-py
Amends the docs concerning multiple label filters
Hmm.. This is supposed to be updated by a web hook. It says that it fired correctly so I'm not sure why the docs weren't updated. I've re-delivered the web hook payload and will watch to see if it updates the docs.
comment created time in a month
Pull request review commentdocker/app
remove references to status & render commands
CNAB too. CNAB specifies three actions which `docker app` provides as commands: * `upgrade` * `uninstall` -Here is an example installing an Application Package, querying its status and+Here is an example installing an Application Package, inspect it and then uninstalling it: ```console $ docker app install examples/hello-world/example-hello-world.dockerapp --name hello Creating network hello_default Creating service hello_hello -$ docker app status hello-ID NAME MODE REPLICAS IMAGE PORTS-0m1wn7jrgkgj hello_hello replicated 1/1 hashicorp/http-echo:latest *:8080->5678/tcp+$ docker app inspect hello+{+ "Metadata": {+ "version": "0.1.0",+ "name": "hello-world",+ "description": "Hello, World!",+ "maintainers": [+ {+ "name": "user",+ "email": "[email protected]"+ }+ ]+ },+ "Services": [+ {+ "Name": "hello",+ "Image": "hashicorp/http-echo",+ "Replicas": 1,+ "Ports": "8080"
@silvin-lubecki is there a way to force the verbose Compose JSON format? I'm worried that if we use short or long format it will be difficult for people to consume.
comment created time in a month
PR opened docker/cli
Backport of #2060
This avoids having redundant output when rendering the compose file once again by adding an option to remove the env_file from the config when already resolved.
pr created time in a month
pull request commentdocker/app
[WIP] Move docker/cli vendoring back to 19.03 branch
undefined: loader.WithDiscardEnvFiles docker/cli@821f5ec not yet in 19.03
@thaJeztah are there plans to add that to 19.03?
comment created time in a month
pull request commentdocker/compose-on-kubernetes
Move from apps/v1beta2 to apps/v1
The change is actually that they dropped the apps/v1beta2 endpoint. The apps/v1 endpoint has been around since Kubernetes 1.9, see this article. As such, we just need to change which endpoint we're calling and don't need to change the Kubernetes client.
comment created time in a month
push eventchris-crone/compose-on-kubernetes
commit sha fc232faf4a7f6ef5fcae2a76608ca3f23c1e0a52
Use upstream yaml.v2 To mitigate against malicious YAML (as described here: https://github.com/kubernetes/kubernetes/issues/83253) we used a patched version of yaml.v2. There is now a fix upstream so we can leverage that. Signed-off-by: Christopher Crone <[email protected]>
push time in a month
PR opened docker/app
Signed-off-by: Christopher Crone [email protected]
<!-- Please make sure you've read and understood our contributing guidelines; https://github.com/docker/app/blob/master/CONTRIBUTING.md
** Make sure all your commits include a signature generated with git commit -s **
For additional information on our contributing process, read our contributing guide https://docs.docker.com/opensource/code/
If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx"
Please provide the following information: -->
- What I did As the YAML library bump is now in the 19.03 branch for the CLI, I've moved our vendoring back to use that.
- How I did it
- Update
Gopkg.toml - Run
make vendor
- How to verify it
- Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: -->
- A picture of a cute animal (not mandatory but encouraged)
pr created time in a month
pull request commentdocker/compose-on-kubernetes
Moved vendoring back to 19.03.
comment created time in a month