Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify cache restore #25186

Merged
merged 5 commits into from
Jul 17, 2019
Merged

Simplify cache restore #25186

merged 5 commits into from
Jul 17, 2019

Conversation

keesschollaart81
Copy link
Contributor

Given the second example in this documentation using the RestoreAndSaveCache task simplifies things a bit. The Save Cache task will be automatically added as a post build step/task!

This is an untested change so please test before merge!

@homeassistant homeassistant added cla-signed small-pr PRs with less than 30 lines. labels Jul 16, 2019
Copy link
Member

@pvizeli pvizeli left a comment

Choose a reason for hiding this comment

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

The issue is, that they uploaded it at the end of the task. That is around 30min later. In that 30 min could other runs already improved by this data. The reason why I didn't add it before

@balloob
Copy link
Member

balloob commented Jul 16, 2019

In that case let's add a comment to the YAML

@keesschollaart81
Copy link
Contributor Author

Great catch! I had not thought about that and it makes total sense! I'll revert the change and add a comment to the YAML for later reference!

azure-pipelines-ci.yml Outdated Show resolved Hide resolved
@pvizeli pvizeli merged commit bc4f91a into home-assistant:dev Jul 17, 2019
KJonline pushed a commit to Rendili/home-assistant that referenced this pull request Jul 17, 2019
…into hive_water_heater

* 'dev' of https://github.com/home-assistant/home-assistant: (49 commits)
  Pin Docker to Debain Stretch (home-assistant#25206)
  Updated frontend to 20190717.0
  Add HmIP-PCBS2, HmIP-PCBS-BAT to Homematic IP Cloud (home-assistant#25201)
  Simplify cache restore (home-assistant#25186)
  Only include target temp if has right support flag (home-assistant#25193)
  Fix issue home-assistant#24495 (home-assistant#25199)
  Handle somfy expired token (home-assistant#25195)
  Add login_method config option to fix login issue with RouterOS Version > 6.43 (home-assistant#25194)
  Add HMIP-FCI / HMIP-FBL / HmIP-BBL (home-assistant#25188)
  [climate] Tweak evohome migration (home-assistant#25187)
  Fix device name customization on ZHA add devices page (home-assistant#25180)
  Upgrade mypy to 0.720, turn on unreachability warnings (home-assistant#25157)
  Use MockConfigEntry (home-assistant#25190)
  Add severe weather sensor to Dark Sky (home-assistant#22701)
  Fix typo in azure-pipelines-ci.yml
  Raise not ready when no data from API is retrieved (home-assistant#25182)
  Improve Nuki lock (home-assistant#22888)
  Delete config.yml (home-assistant#25181)
  Upgrade luftdaten to 0.6.2 (home-assistant#25177)
  Upgrade Mastodon.py to 1.4.5 (home-assistant#25176)
  ...

# Conflicts:
#	homeassistant/components/hive/water_heater.py
@lock lock bot locked and limited conversation to collaborators Jul 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed small-pr PRs with less than 30 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants