Skip to content
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

mobile/ci: Fix workflow dependencies #27592

Merged
merged 1 commit into from
May 24, 2023
Merged

Conversation

phlax
Copy link
Member

@phlax phlax commented May 24, 2023

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@phlax
Copy link
Member Author

phlax commented May 24, 2023

not sure if this was a bad merge, or how it got through CI, i had to fix the same on the pr for the android build

@phlax
Copy link
Member Author

phlax commented May 24, 2023

/retest mobile

@phlax
Copy link
Member Author

phlax commented May 24, 2023

/retest mobile

@phlax
Copy link
Member Author

phlax commented May 24, 2023

@alyssawilk afaict there is another permafail here:

external/envoy/test/integration/ssl_utility.cc:12:10: fatal error: 'test/config/utility.h' file not found
#include "test/config/utility.h"
         ^~~~~~~~~~~~~~~~~~~~~~~

https://github.com/envoyproxy/envoy/actions/runs/5065551466/jobs/9094259070?pr=27592#step:5:102

the changes in this PR are required i think so i will push it through, and will try and track where it started

@phlax phlax merged commit 2123fa9 into envoyproxy:main May 24, 2023
reskin89 pushed a commit to reskin89/envoy that referenced this pull request Jul 11, 2023
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.

1 participant