Skip to content

Conversation

fi
- name: Publish ${{ inputs.plugin_name }}
if: needs.check-uploaded.outputs.plugin_uploaded == 'false' || github.event_name == 'workflow_dispatch'
Copy link
Contributor

Choose a reason for hiding this comment

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

github.event_name == 'workflow_dispatch'
this line is responsible for publishing plugin manually
As you can see, the only way to trigger _publish-plugin.yml is workflow_call, but there is no such event_name as workflow_call, event_name is always inherited from calling workflow, in that scenerio it will be always, workflow_dispatch from publish-plugin.yml, that can be triggered only in that way

Copy link
Member Author

Choose a reason for hiding this comment

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

Right, but even in case of manually running a workflow, we should still check if the plugin was not published already. Because if it was, the workflow will just fail, unless that was the intended behavior?

The problem is, _publish-plugin workflow is also reused in nightly workflow which is also triggered manually. If any step of nightly workflow fails after uploading plugins, then due to this line, you cannot re-run it because on subsequent runs it will fail as it will try to upload plugin again (which registry doesn't allow).

@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from 6c35a9d to ca0a82a Compare July 18, 2025 11:19
@cptartur cptartur force-pushed the spr/master/fa08970b branch 2 times, most recently from 87da6e1 to 4ecadbb Compare July 18, 2025 13:44
@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from ca0a82a to 4f3e920 Compare July 18, 2025 13:44
@cptartur cptartur force-pushed the spr/master/fa08970b branch from 4ecadbb to 5587a2f Compare July 18, 2025 13:47
@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from 4f3e920 to d09878a Compare July 18, 2025 13:47
@cptartur cptartur force-pushed the spr/master/fa08970b branch from 5587a2f to 677adac Compare July 18, 2025 13:55
@cptartur cptartur marked this pull request as ready for review July 18, 2025 15:12
@cptartur cptartur requested a review from a team as a code owner July 18, 2025 15:12
@cptartur cptartur requested review from franciszekjob and MKowalski8 and removed request for a team July 18, 2025 15:12
@cptartur cptartur requested review from ddoktorski and removed request for MKowalski8 July 18, 2025 15:12
@cptartur cptartur force-pushed the spr/master/fa08970b branch from 677adac to 30098cd Compare July 21, 2025 11:57
@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from 0085c56 to d114d09 Compare July 21, 2025 11:57
@cptartur cptartur force-pushed the spr/master/fa08970b branch 3 times, most recently from 3f90191 to 9c07719 Compare July 24, 2025 14:42
@cptartur cptartur mentioned this pull request Jul 24, 2025
@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from 1bb6493 to f77ab9e Compare July 24, 2025 14:45
@cptartur cptartur force-pushed the spr/master/fa08970b branch from 9c07719 to 5dbf277 Compare July 24, 2025 14:45
@cptartur cptartur force-pushed the spr/master/9d3bc55c branch from f77ab9e to 7bb7b2a Compare July 25, 2025 13:55
@cptartur cptartur force-pushed the spr/master/fa08970b branch from 5dbf277 to cf5fcb2 Compare July 25, 2025 13:55
@cptartur
Copy link
Member Author

✓ Commit merged in pull request #3559

@cptartur cptartur closed this Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants