Pantheon.yml update fails sometimes

[generalredneck]

Uri Frazier
6 days ago
Has anyone run into this error in CircleCi while updating pantheon.yml so that the site uses Solr8?
'8# Workaround change to pantheon.yml to force envrionment update' is not one of [3, 8]
(Reference: https://docs.pantheon.io/pantheon-yml)
3 files

Dave Hansen-Lange
6 days ago
I might’ve heard something where there’s a trick to get Circle-managed multidevs to recognize changes to pantheon.yml.
@genred
Would know.

Allan Chappell
6 days ago
yeah... if you are using Project CI... it's built in

Uri Frazier
6 days ago
It worked to get PHP updated (php_version), but it's not liking this search change. (edited)

Allan Chappell
6 days ago
this is the part:
https://github.com/fourkitchens/pots/blob/main/scripts/pantheon/deploy#L90-L93

deploy
if [[ ! -z "$PANTHEON_YML_DIFF" || ! -z "$PANTHEON_UPSTREAM_YML_DIFF" ]]; then
echo "# Workaround change to pantheon.yml to force envrionment update" >> pantheon.yml
terminus -n build:env:push "$TERMINUS_SITE.$TERMINUS_ENV" --message="Workaround change to pantheon.yml to force envrionment update"
fi
https://github.com/[fourkitchens/pots](https://github.com/fourkitchens/pots)|fourkitchens/potsfourkitchens/pots | Added by GitHub

Dave Hansen-Lange
6 days ago
@genred
See the screenshot. That part seems to be erroring out.

Allan Chappell
6 days ago
yeah I see that

Allan Chappell
6 days ago
ok so pantheon.yml doesn't end with an endline

Allan Chappell
6 days ago
all linux files should have an eol at the end of the file

Allan Chappell
6 days ago
that's best practice

Uri Frazier
6 days ago
Ahh - thanks!
@genred

• I'll give that a try.

Allan Chappell
6 days ago
we can make an issue out of that (edited)

Allan Chappell
6 days ago
cause I could always add an EOL in there to make sure

Uri Frazier
6 days ago
That fixed the CircleCi error - thanks again!
@genred
&
@Dave
for flagging Allan.