Skip to content

Commit 9e8de6d

Browse files
bwannmeta-codesync[bot]
authored andcommitted
fb_helpers: interface_change_allowed regexp fix (facebook#342) (facebook#343)
Test Plan: Imported from GitHub, without a `Test Plan:` line. Ran chef, confirmed no pending network change triggered: ``` [root@twshared22960.15.ftw3 ~]# ls -la /run/chef/ total 0 drwxrwxrwt 2 root root 40 Nov 13 01:17 . drwxr-xr-x 74 root root 1860 Nov 26 16:46 .. ``` Reviewed By: dafyddcrosby Differential Revision: D87958589 Pulled By: joshuamiller01 fbshipit-source-id: 8e28e9c8743ff7d383c074238022505f2e3d04ea
1 parent 2768e87 commit 9e8de6d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

cookbooks/fb_helpers/libraries/node_methods.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1451,7 +1451,7 @@ def interface_change_allowed?(interface)
14511451
else
14521452
return self.nw_changes_allowed? ||
14531453
['ip6tnl0', 'tunlany0', 'tunl0'].include?(interface) ||
1454-
interface.match(Regexp.new('^tunlany\d+:\d+'))
1454+
interface.match?(Regexp.new('^tunlany\d+:\d+'))
14551455
end
14561456
end
14571457

0 commit comments

Comments
 (0)