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

GHA zip workflow: add manual trigger #7450

Merged
merged 6 commits into from
Feb 4, 2021
Merged

Conversation

maksymmalyhin
Copy link
Contributor

@maksymmalyhin maksymmalyhin commented Feb 3, 2021

An attempt to add a manual trigger for zip GHA workflow.

I was not able to test the trigger itself as the "Run workflow" button didn't appear for me. I guess the change has to land to the default branch to be picked up by GHA.

@google-cla google-cla bot added the cla: yes label Feb 3, 2021
@maksymmalyhin
Copy link
Contributor Author

The updated package step triggered the correct command for non-manual run: https://github.com/firebase/firebase-ios-sdk/runs/1826829858?check_suite_focus=true#step:4:12
Further packaging was cancelled.

@maksymmalyhin maksymmalyhin marked this pull request as ready for review February 4, 2021 00:06
custom_spec_repos:
description: 'Custom Podspec repos'
required: true
default: 'https://cpdc-internal.git.corp.google.com/firebase.git'
Copy link
Member

Choose a reason for hiding this comment

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

I don't think GHA has access to https://cpdc-internal.git.corp.google.com. I believe we need the whole staging process to SpecsStaging or another public repo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I would prefer moving release staging to GitHub to simplify GHA access control. I think we can address it in following PRs. For now I would like to test the manual trigger in general.

@@ -37,7 +44,7 @@ jobs:
- name: ZipBuildingTest
run: |
mkdir -p zip_output_dir
Copy link
Member

Choose a reason for hiding this comment

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

Add an xcode-select command to Xcode 12.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would prefer to add it in a separate PR. With the manual trigger testing will be so much easier :).

@maksymmalyhin maksymmalyhin merged commit 7ecc4a7 into master Feb 4, 2021
@maksymmalyhin maksymmalyhin deleted the mm/gha-zip-manual branch February 4, 2021 00:22
@firebase firebase locked and limited conversation to collaborators Mar 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants