-
Notifications
You must be signed in to change notification settings - Fork 31
Release pixel cleanup #2070
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
base: bartek/ci-release-pixel
Are you sure you want to change the base?
Release pixel cleanup #2070
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: iOS Workflow Missing New Pixel Format
The iOS release workflow removes the pixel sending step without replacing it with the new format. This contradicts the PR description's intent to replace old pixel steps and creates an inconsistency with the macOS workflow, which was updated to use the new format.
.github/workflows/ios_tag_release_update_asana.yml#L233-L244
apple-browsers/.github/workflows/ios_tag_release_update_asana.yml
Lines 233 to 244 in f777316
template_name:"${{ steps.asana-templates.outputs.release-task-comment-template }}" | |
- name: Create Asana notification task | |
if: github.event.inputs.release-type != null && github.event.inputs.release-type != 'internal' | |
continue-on-error: true | |
uses: ./.github/actions/send-release-notification-task | |
with: | |
platform: ios | |
tag: ${{ env.TAG }} | |
asana-project: ${{ vars.APPLE_DEPLOYMENTS_ASANA_PROJECT_ID }} | |
asana-section: ${{ vars.APPLE_DEPLOYMENTS_SECTION_ID }} | |
asana-access-token: ${{ secrets.ASANA_ACCESS_TOKEN }} |
- name: Send release annotation pixel | ||
if: github.event.inputs.release-type != null && github.event.inputs.release-type != 'internal' | ||
- name: Send Release Pixel | ||
if: success() && github.event.inputs.release-type != null && github.event.inputs.release-type != 'internal' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What I wonder is if we want to have it for internals as well:
1/ we can have only for public
2/ we can have for all
3/ we can have two pixels, one for internal and one for public releases
I’m sort of leaning towards 3, as it gives us most flexibility around how to use these pixels, but it’s a bt more tricky to send (e.g. we’d have internal, that then gets promoted to public).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah good question. If we do want internal pixels (and we may as well), then I agree option 3 is suitable. I think this will be doable by keeping this pixel as-is as the final release, but updating the other workflow file to send the internal pixel - let me take a go at that on Thursday.
This PR has been inactive for more than 7 days and will be automatically closed 7 days from now. |
Task/Issue URL: https://app.asana.com/1/137249556945/task/1211470395820605?focus=true
Tech Design URL:
CC:
Description
This PR removes the old macOS and iOS release pixels, and replaces them with the new format.
Testing Steps
Impact and Risks
What could go wrong?
Quality Considerations
Notes to Reviewer
Internal references:
Definition of Done | Engineering Expectations | Tech Design Template
Note
Remove old release-annotation pixel action and iOS usage; switch macOS release workflow to
send-pixel
with new pixel name and stricter run condition./.github/workflows/macos_publish_dmg_release.yml
:send-release-annotation-pixel
step withsend-pixel
usingpixel-name: m_ci_apple_macos_release_version_${{ steps.verify-tag.outputs.release-version }}
.success()
to the step condition; rename step to "Send Release Pixel"./.github/workflows/ios_tag_release_update_asana.yml
:/.github/actions/send-release-annotation-pixel/action.yml
(legacy composite action).Written by Cursor Bugbot for commit f777316. This will update automatically on new commits. Configure here.