feat(lua actions): add a flag to Include builtin actions with resource overrides

[ashutosh16]

fixes #19707

To test:
add the below config to the configmap

data:
 resource.customizations.actions.argoproj.io_Rollout: |
 mergeBuiltinActions: true
  discovery.lua: |
    actions = {}
    actions["test-action"] = {}
    return actions
  definitions:
  - name: test-action
    action.lua: |
      return obj

Verified the following:

• existing behaviour with overrides and prebuilt actions
• with mergeBuiltinActions set to true. action contains upstream action and override actions

Screen.Recording.2024-09-03.at.14.25.41.mov

Checklist:

• Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
• The title of the PR states what changed and the related issues number (used for the release note).
• The title of the PR conforms to the Toolchain Guide
• I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
• I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
• Does this PR require documentation updates?
• I've updated documentation as required by this PR.
• I have signed off all my commits as required by DCO
• I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
• My build is green (troubleshooting builds).
• My new feature complies with the feature status guidelines.
• I have added a brief description of why this PR is necessary and/or what this PR solves.
• Optional. My organization is added to USERS.md.
• Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

[bunnyshell]

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

• 🚀 /bns:deploy to deploy the environment

[bunnyshell]

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

• 🚀 /bns:deploy to deploy the environment

[codecov]

Codecov Report

Attention: Patch coverage is 72.91667% with 13 lines in your changes missing coverage. Please review.

Project coverage is 55.83%. Comparing base (878494f) to head (b43d05b).
Report is 448 commits behind head on master.

Files with missing lines	Patch %	Lines
util/lua/lua.go	73.33%	7 Missing and 5 partials ⚠️
server/application/application.go	66.66%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #19708      +/-   ##
==========================================
- Coverage   55.84%   55.83%   -0.01%     
==========================================
  Files         320      320              
  Lines       44316    44329      +13     
==========================================
+ Hits        24747    24751       +4     
- Misses      17012    17019       +7     
- Partials     2557     2559       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[leoluz]

Please check my comments

[agaudreault]

FYI, take a look at #16905 (comment).

The merge might be conflicting or result in an undefined behavior if glob are used.

[ashutosh16]

FYI, take a look at #16905 (comment).

The merge might be conflicting or result in an undefined behavior if glob are used.

I don't see the glob is used to determine resource action.

[agaudreault]

LGTM

accepted