Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

update script for pushing iamges to docker hub #302

Merged
merged 3 commits into from
Apr 3, 2019

Conversation

27149chen
Copy link
Contributor

@27149chen 27149chen commented Apr 2, 2019

Signed-off-by: 27149chen 7991675+27149chen@users.noreply.github.com


This change is Reviewable

Signed-off-by: 27149chen <7991675+27149chen@users.noreply.github.com>
@27149chen 27149chen added this to the Helm chart support milestone Apr 2, 2019
@27149chen 27149chen self-assigned this Apr 2, 2019
@coveralls
Copy link

coveralls commented Apr 2, 2019

Coverage Status

Coverage remained the same at 59.175% when pulling aa2bc31 on feature/update_push_dockerhub_script into 8994dcf on dev.

Copy link
Contributor

@shay-berman shay-berman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 1 files reviewed, 3 unresolved discussions (waiting on @27149chen and @shay-berman)


scripts/ci/helper_to_push_ubiquity_images_and_manifests.sh, line 56 at r1 (raw file):

ubiquity_flex_envs="in_UBIQUITY_K8S_FLEX_IMAGE_AMD64 out_UBIQUITY_K8S_FLEX_IMAGE_AMD64 in_UBIQUITY_K8S_FLEX_IMAGE_PPC64LE out_UBIQUITY_K8S_FLEX_IMAGE_PPC64LE in_UBIQUITY_K8S_FLEX_IMAGE_S390X out_UBIQUITY_K8S_FLEX_IMAGE_S390X out_UBIQUITY_K8S_FLEX_IMAGE_MULTIARCH"
ubiquity_flex_sidecar_envs="in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_AMD64 out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_AMD64 in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_PPC64LE out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_PPC64LE in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_S390X out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_S390X out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_MULTIARCH"
ubiquity_helm_envs="in_UBIQUITY_K8S_HELM_IMAGE_AMD64 out_UBIQUITY_K8S_HELM_IMAGE_AMD64 in_UBIQUITY_K8S_HELM_IMAGE_PPC64LE out_UBIQUITY_K8S_HELM_IMAGE_PPC64LE in_UBIQUITY_K8S_HELM_IMAGE_S390X out_UBIQUITY_K8S_HELM_IMAGE_S390X out_UBIQUITY_K8S_HELM_IMAGE_MULTIARCH"

the word helm here is a bit confusing. if u can please:

  1. rename it to ubiquity_helmhook_envs.
  2. rename all the env names HELM -> HELMHOOK

and of course change other places when used.


scripts/ci/helper_to_push_ubiquity_images_and_manifests.sh, line 70 at r1 (raw file):

push_arch_images_and_create_manifest_for_app "Ubiquity"              $in_UBIQUITY_IMAGE_AMD64                 $out_UBIQUITY_IMAGE_AMD64                          $in_UBIQUITY_IMAGE_PPC64LE                 $out_UBIQUITY_IMAGE_PPC64LE                          $in_UBIQUITY_IMAGE_S390X                 $out_UBIQUITY_IMAGE_S390X                          $out_UBIQUITY_IMAGE_MULTIARCH                  $TAG_LATEST

Lou
looks like some indentation change in these 5 lines, which should not be touched.
Please push commit that don't touch these lines, because its not related to the helm\side car.


scripts/ci/helper_to_push_ubiquity_images_and_manifests.sh, line 76 at r1 (raw file):

push_arch_images_and_create_manifest_for_app "Ubiquity provisioner"   $in_UBIQUITY_K8S_PROVISIONER_IMAGE_AMD64   $out_UBIQUITY_K8S_PROVISIONER_IMAGE_AMD64          $in_UBIQUITY_K8S_PROVISIONER_IMAGE_PPC64LE   $out_UBIQUITY_K8S_PROVISIONER_IMAGE_PPC64LE          $in_UBIQUITY_K8S_PROVISIONER_IMAGE_S390X   $out_UBIQUITY_K8S_PROVISIONER_IMAGE_S390X          $out_UBIQUITY_K8S_PROVISIONER_IMAGE_MULTIARCH    $TAG_LATEST
push_arch_images_and_create_manifest_for_app "Ubiquity flex"          $in_UBIQUITY_K8S_FLEX_IMAGE_AMD64          $out_UBIQUITY_K8S_FLEX_IMAGE_AMD64                 $in_UBIQUITY_K8S_FLEX_IMAGE_PPC64LE          $out_UBIQUITY_K8S_FLEX_IMAGE_PPC64LE                 $in_UBIQUITY_K8S_FLEX_IMAGE_S390X          $out_UBIQUITY_K8S_FLEX_IMAGE_S390X                 $out_UBIQUITY_K8S_FLEX_IMAGE_MULTIARCH           $TAG_LATEST
push_arch_images_and_create_manifest_for_app "Ubiquity flex sidecar"  $in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_AMD64  $out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_AMD64         $in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_PPC64LE  $out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_PPC64LE         $in_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_S390X  $out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_S390X         $out_UBIQUITY_K8S_FLEX_SIDECAR_IMAGE_MULTIARCH   $TAG_LATEST

these r the 2 lines that should be added.
which is ok.

Signed-off-by: 27149chen <7991675+27149chen@users.noreply.github.com>
Signed-off-by: 27149chen <7991675+27149chen@users.noreply.github.com>
@27149chen 27149chen merged commit 3f22f0e into dev Apr 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants