vmware-archive / pcf-pipelines

PCF Pipelines
Apache License 2.0
158 stars 282 forks source link

apply-updates regulator job doesn't regulate anything #201

Closed krishicks closed 6 years ago

krishicks commented 7 years ago

https://github.com/pivotal-cf/pcf-pipelines/blob/dbb274353035b0d54d04057af9904a8d8e94e9fe/apply-updates/pipeline.yml#L26

The apply-updates regulator job is currently being triggered on the schedule alone; in this configuration the job has no meaning.

See https://github.com/concourse/concourse/issues/945 for an explanation of the regulator concept.

cf-gitbot commented 7 years ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

krishicks commented 7 years ago

The changes made to this pipeline remove the regulator, but still get the implementation incorrect.

In the event that there is an updated stemcell available it will download that stemcell, and upload it. The apply-changes job will then run apply changes.

In the event that there is not an updated stemcell available, it will simply state "No new stemcells available, skipping upload." and exit successfully, meaning apply-updates will still run. This isn't free; OpsMgr will take time trying to apply changes even when there are none.

Also, the default configuration means this Apply Changes will run once per day, every day, whether there are updates or not.

abbyachau commented 7 years ago

Hi @krishicks thanks for that feedback. Two stories have been created to address your feedback: https://www.pivotaltracker.com/story/show/152622902 and https://www.pivotaltracker.com/story/show/152622921.

abbyachau commented 6 years ago

Hi @krishicks instead of leaving this issue open so that we could get to the issues you described, we've decided to close it and add a Known Issues section to the Apply Updates README. Thanks again for the feedback.