fix: remove unnecessary apt lock waits

[jackfrancis]

Reason for Change:

This PR removes superfluous "wait for no held apt locks" after apt operations have already completed. Instead we are waiting only before attempting an apt operation. This configuration in concert will promote good apt race condition avoidance without unnecessarily blocking CSE progression (for example, the next operation after an apt operation is probably not another apt operation, and so waiting for apt locks to be released means we're delaying that next operation unnecessarily).

Issue Fixed:

Requirements:

• uses conventional commit messages
• includes documentation
• adds unit tests
• tested upgrade from previous version

Notes:

[codecov]

Codecov Report

Merging #3049 into master will increase coverage by 0.04%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##           master    #3049      +/-   ##
==========================================
+ Coverage   70.58%   70.62%   +0.04%     
==========================================
  Files         145      145              
  Lines       25105    25147      +42     
==========================================
+ Hits        17721    17761      +40     
- Misses       6281     6283       +2     
  Partials     1103     1103              

Impacted Files	Coverage Δ
pkg/engine/templates_generated.go	31.57% <ø> (ø)
pkg/helpers/helpers.go	79.83% <88.23%> (+1.40%)	⬆️
pkg/api/vlabs/validate.go	79.03% <100.00%> (+0.40%)	⬆️
pkg/api/types.go	94.05% <0.00%> (ø)
pkg/api/vlabs/types.go	74.31% <0.00%> (ø)
pkg/api/convertertoapi.go	93.63% <0.00%> (+0.02%)	⬆️
pkg/api/converterfromapi.go	93.82% <0.00%> (+0.02%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a34150...39e3a1f. Read the comment docs.

[codecov]

Codecov Report

Merging #3049 into master will increase coverage by 0.04%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##           master    #3049      +/-   ##
==========================================
+ Coverage   70.58%   70.62%   +0.04%     
==========================================
  Files         145      145              
  Lines       25105    25147      +42     
==========================================
+ Hits        17721    17761      +40     
- Misses       6281     6283       +2     
  Partials     1103     1103              

Impacted Files	Coverage Δ
pkg/engine/templates_generated.go	31.57% <ø> (ø)
pkg/helpers/helpers.go	79.83% <88.23%> (+1.40%)	⬆️
pkg/api/vlabs/validate.go	79.03% <100.00%> (+0.40%)	⬆️
pkg/api/types.go	94.05% <0.00%> (ø)
pkg/api/vlabs/types.go	74.31% <0.00%> (ø)
pkg/api/convertertoapi.go	93.63% <0.00%> (+0.02%)	⬆️
pkg/api/converterfromapi.go	93.82% <0.00%> (+0.02%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a34150...39e3a1f. Read the comment docs.

[mboersma]

/lgtm

Rationale makes sense to me as well. Simpler is better too.