Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Fix some steps in README.md to install flux by helm #2532

Merged
merged 1 commit into from
Nov 8, 2019
Merged

Fix some steps in README.md to install flux by helm #2532

merged 1 commit into from
Nov 8, 2019

Conversation

at-ishikawa
Copy link
Contributor

Summary

Update next steps in README.md for instruction to install flux by helm.

  1. Update github URL format for https
  2. Add namespace to create k8s secret

Error details

  1. If we see the logs on k8s pods, next logs can be seen (USERNAME, TOKEN, and REPOSITORY are not the real values and they should be real values).

    ts=2019-10-14T05:47:42.358161033Z caller=loop.go:101 component=sync-loop err="git repo not ready: git clone --mirror: fatal: unable to access 'https://USERNAME:[email protected]:at-ishikawa/REPOSITORY.git/': URL using bad/illegal format or missing URL, full output:\n Cloning into bare repository '/tmp/flux-gitclone859872933'...\nfatal: unable to access 'https://USERNAME:[email protected]:at-ishikawa/REPOSITORY.git/': URL using bad/illegal format or missing URL\n"
    
  2. If we do not add namespace in flux secret, we can get following events on the result of kubectl describe pods.

    # kubectl -n flux describe pods -l app=flux
    ## A lot of details
    Events:
      Type     Reason     Age               From                                                       Message
      ----     ------     ----              ----                                                       -------
    ## A lot of events
      Warning  Failed     1s (x9 over 94s)  kubelet, gke-cluster-preemptible-node-pool-9d547b9e-8lj1  Error: secrets "flux-git-auth" not found
    

Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @at-ishikawa

@stefanprodan stefanprodan added the docs Issue or PR relates to documentation label Oct 20, 2019
@2opremio 2opremio merged commit 9199738 into fluxcd:master Nov 8, 2019
@2opremio 2opremio added this to the 1.16.0 milestone Nov 21, 2019
@at-ishikawa at-ishikawa deleted the update-readme branch November 30, 2019 03:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Issue or PR relates to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants