Skip to content

Conversation

@wking
Copy link
Member

@wking wking commented Aug 17, 2018

Avoiding:

Invoking installer ...
time="2018-08-17T18:05:11Z" level=fatal msg="failed to get configuration from file \"inputs.yaml\": inputs.yaml is not a valid config file: yaml: line 75: found unexpected ':'"

due to:

$ curl -s https://storage.googleapis.com/origin-ci-test/pr-logs/pull/openshift_installer/142/pull-ci-origin-installer-e2e-aws/523/artifacts/e2e-aws/installer/inputs.yaml | sed -n '75p;76p'
  # Example: `{ "key" = "value", "foo" = "bar" }`
  extraTags: {"expirationDate" = "2018-08-17T22:05+0000"}

The typo is from f64b8ea (#1103).

Avoiding [1]:

  Invoking installer ...
  time="2018-08-17T18:05:11Z" level=fatal msg="failed to get configuration from file \"inputs.yaml\": inputs.yaml is not a valid config file: yaml: line 75: found unexpected ':'"

due to:

  $ curl -s https://storage.googleapis.com/origin-ci-test/pr-logs/pull/openshift_installer/142/pull-ci-origin-installer-e2e-aws/523/artifacts/e2e-aws/installer/inputs.yaml | sed -n '75p;76p'
    # Example: `{ "key" = "value", "foo" = "bar" }`
    extraTags: {"expirationDate" = "2018-08-17T22:05+0000"}

The typo is from f64b8ea (cluster-launch-installer-e2e: Set
expirationDate tags, 2018-07-26, openshift#1103).

[1]: https://storage.googleapis.com/origin-ci-test/pr-logs/pull/openshift_installer/142/pull-ci-origin-installer-e2e-aws/523/build-log.txt
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 17, 2018
@smarterclayton smarterclayton added the lgtm Indicates that a PR is ready to be merged. label Aug 17, 2018
@openshift-merge-robot openshift-merge-robot merged commit cff952c into openshift:master Aug 17, 2018
@wking wking deleted the installer-extra-tags-equal-typo branch August 17, 2018 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants