apt_repository LWRP does not run apt-get update when new file is created

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Unknown Unknown
  • Resolution: Fixed
  • Component/s: apt
  • Labels:
    None
  • Environment:

    chef 10.10
    ubuntu precise

Description

On line 100 of the apt_repository LWRP

if repo_file.updated_by_last_action?

is nil.

This is because file#action_create sets updated_by_last_action(true) when the file is created but then updated_by_last_action(nil) because permissions do not change (enforce_ownership_and_permissions.rb line 34)

Using

if repo_file.updated?

works

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. COOK-1382 Trigger APT update immediately after adding a new repository

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Gilles Devaux added a comment -

Pull request https://github.com/opscode/cookbooks/pull/405

Show
Gilles Devaux added a comment - Pull request https://github.com/opscode/cookbooks/pull/405
Hide
Permalink
jtimberman added a comment -

This is duplicated by COOK-1382. However, the pull request above is against the deprecated monolithic https://github.com/opscode/cookbooks repository, and the current repository is https://github.com/opscode-cookbooks/apt.

Show
jtimberman added a comment - This is duplicated by COOK-1382 . However, the pull request above is against the deprecated monolithic https://github.com/opscode/cookbooks repository, and the current repository is https://github.com/opscode-cookbooks/apt .
Hide
Permalink
Daniel DeLeo added a comment -

This is caused by a bug in Chef in 10.10, see CHEF-3135

Show
Daniel DeLeo added a comment - This is caused by a bug in Chef in 10.10, see CHEF-3135

People

  • Assignee:
    jtimberman
    Reporter:
    Gilles Devaux
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: