Skip to content

fix federation integration workflows #1687

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 28, 2023

Conversation

dariuszkuc
Copy link
Collaborator

📝 Description

Conditional steps do not work in workflows that are invoked from other workflows :(

🔗 Related Issues

Conditional steps do not work in workflows that are invoked from other workflows :(
@dariuszkuc dariuszkuc added the type: tool update CI, code coverage, or linter updates label Feb 24, 2023
@dariuszkuc
Copy link
Collaborator Author

We need pull_request_target event to be able to pass secrets (needed to comment on PRs) to PRs made from forks. One way to work around this limitation is to have separate workflow that triggers based on completion of previous workflow but that is somewhat PITA to setup (e.g. see example)

runs-on: ubuntu-latest
defaults:
run:
working-directory: integration/federation-compatibility
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sadly GH action defaults/step config for working-directory only work for run steps... when using custom actions they have to implement their own working directory logic (hence the duplicate param below).

@dariuszkuc
Copy link
Collaborator Author

fed compatibility failure is due to invalid workflow -> it runs workflow from master against the code from the fork (so it cannot find the referenced file).

@dariuszkuc dariuszkuc enabled auto-merge (squash) February 24, 2023 20:32
@samuelAndalon samuelAndalon self-requested a review February 28, 2023 18:59
@dariuszkuc dariuszkuc merged commit 37338f4 into ExpediaGroup:master Feb 28, 2023
@dariuszkuc dariuszkuc deleted the fix_fed_workflows branch February 28, 2023 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: tool update CI, code coverage, or linter updates
Development

Successfully merging this pull request may close these issues.

2 participants