Updated bb values.yaml to have a global value for image pull policy for all core packages

Package Merge Request

Package Changes

• Updated chart/values.yaml to include a global value for imagePullPolicy for all core packages (imagePullPolicy:{})
• Updated the values.yaml for all the core package template/spec in BB to make sure the global value is passed down to all core packages
• Updated image_pull_policy.md doc to show how to use the new global value for the core packages

For Issue

Closes #897 (closed)

added statusreview label

requested review from @micah.nagel

added 1 commit

• e8ce3678 - Fixed a typo in image_pull_policy doc

Compare with previous version

added 1 commit

• 9994a792 - Updated the value for global imagePullPolicy

Compare with previous version

resolved all threads

added 1 commit

• cdf9e449 - upated documentation for global image pull policy

Compare with previous version

Deploying now to make sure this works. Changes look good to me, just a few docs suggestions.

added 1 commit

• 16577471 - Fixed typo for imagePullPolicy

Compare with previous version

added 1 commit

• ab8e8833 - Updated docs for global image pull policy

Compare with previous version

added 1 commit

• cfe3e5ec - Doc updates for image pull policy

Compare with previous version

resolved all threads

resolved all threads

added 1 commit

• 55324c12 - Update to image pull policy doc

Compare with previous version

requested review from @BrandenCobb and @ryan.j.garcia

Tested on my cluster, everything expected had pull policy set to whatever I set in my values

kubectl get po -A -o go-template='{{range $pod := .items}}{{range $container := $pod.spec.containers}}{{$pod.metadata.name}}{{" : "}}{{$container.image}}{{" : "}}{{$container.imagePullPolicy}}{{"\n"}}{{end}}{{end}}'

for your go-template enjoyment @BrandenCobb @ryan.j.garcia , podname : containerimage : pullpolicy. There are some pods we don't have this set on...jaeger, couple monitoring things, and maybe some others. Everything we know of overrides for works though.

approved this merge request

added 9 commits

• 55324c12...264fec1b - 8 commits from branch master
• aeb376b3 - Merge branch 'master' into test

Compare with previous version

approved this merge request

approved this merge request

approved this merge request

merged

mentioned in commit 63e09c4b

changed milestone to %1.23.0