Comment 3 for bug 1622899

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to k8s-docker-suite-app-murano (master)

Reviewed: https://review.openstack.org/383657
Committed: https://git.openstack.org/cgit/openstack/k8s-docker-suite-app-murano/commit/?id=82056d980f9ce912ce7352e3d80e3333d5e02501
Submitter: Jenkins
Branch: master

commit 82056d980f9ce912ce7352e3d80e3333d5e02501
Author: Peter Razumovsky <email address hidden>
Date: Fri Oct 7 14:00:45 2016 +0300

    Fix name pattern for DockerApp

    Currently name pattern for DockerApp allows to use underlines, dots and
    doesn't limit name length (at least that's described in field description).
    This occurs validation error with some regexp. Kubernetes docs [1] indicates,
    that container name should be DNS_LABEL, which described in [2]. Also,
    kubernetes code [3] contains regexp and length limit for DNS_LABEL
    pattern.

    [1] http://kubernetes.io/docs/user-guide/pods/multi-container/#containers

    [2] http://kubernetes.io/docs/user-guide/labels/#syntax-and-character-set

    [3] https://github.com/kubernetes/kubernetes/blob/8fd414537b5143ab039cb910590237cabf4af783/pkg/util/validation/validation.go#L88-L89

    Change-Id: Icb2ed2b00afe15bd547c01895beedd5ff02b0b0d
    Closes-bug: #1622899