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

[v.1.13.0] Release Tracker #86312

Closed
atalman opened this issue Oct 5, 2022 · 60 comments
Closed

[v.1.13.0] Release Tracker #86312

atalman opened this issue Oct 5, 2022 · 60 comments
Labels
oncall: releng In support of CI and Release Engineering triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Milestone

Comments

@atalman
Copy link
Contributor

atalman commented Oct 5, 2022

We cut a release branch for the 1.13.0 release.

Our plan from this point from this point is roughly:

  • Phase 1 (until 10/21): work on finalizing the release branch
  • Phase 2 (after 10/21): perform extended integration/stability/performance testing based on Release Candidate builds.

This issue is for tracking cherry-picks to the release branch.

Cherry-Pick Criteria

Phase 1 (until 10/21/22):

Only low-risk changes may be cherry-picked from master:

  1. Fixes to regressions against the most recent minor release (e.g. 1.12 for 1.13 release; see module: regression issue list)
  2. Critical fixes for: silent correctness, backwards compatibility, crashes, deadlocks, (large) memory leaks
  3. Fixes to new features introduced in the most recent minor release (e.g. 1.12.0 for 1.13 release)
  4. Test/CI fixes
  5. Documentation improvements
  6. Compilation fixes or ifdefs required for different versions of the compilers or third-party libraries
  7. Release branch specific changes (e.g. change version identifiers)

Any other change requires special dispensation from the release managers (currently @atalman, @weiwangmeta, @izaitsevfb, @malfet ). If this applies to your change please write "Special Dispensation" in the "Criteria Category:" template below and explain.

Phase 2 (after 10/21/22):

Note that changes here require us to rebuild a Release Candidate and restart extended testing (likely delaying the release). Therefore, the only accepted changes are Release-blocking critical fixes for: silent correctness, backwards compatibility, crashes, deadlocks, (large) memory leaks

Changes will likely require a discussion with the larger release team over VC or Slack.

Cherry-Pick Process

  1. Ensure your PR has landed in master. This does not apply for release-branch specific changes (see Phase 1 criteria).

  2. Create (but do not land) a PR against the release branch.

    # Find the hash of the commit you want to cherry pick
    # (for example, abcdef12345)
    git log
    
    git fetch origin release/1.13
    git checkout release/1.13
    git cherry-pick abcdef12345
    
    # Submit a PR based against 'release/1.13' either:
    # via the GitHub UI
    git push my-fork
    
    # via the GitHub CLI
    gh pr create --base release/1.13
  3. Make a request below with the following format:

Link to landed master PR (if applicable):
* 

Link to release branch PR:
* 

Criteria Category:
* 
  1. Someone from the release team will reply with approved / denied or ask for more information.
  2. If approved, someone from the release team will merge your PR once the tests pass. Do not land the release branch PR yourself.

NOTE: Our normal tools (ghstack / ghimport, etc.) do not work on the release branch.

Please note HUD Link with branch CI status and link to the HUD to be provided here.
HUD

Versions

1.13.0

@atalman atalman added oncall: releng In support of CI and Release Engineering triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module labels Oct 5, 2022
@atalman atalman added this to the 1.13.0 milestone Oct 5, 2022
@atalman atalman pinned this issue Oct 5, 2022
@zou3519
Copy link
Contributor

zou3519 commented Oct 5, 2022

Link to landed master PR (if applicable):

  • n/a

Link to release branch PR:

Criteria Category:

  • Release branch specific changes (remove prototype feature)

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 6, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Fix full jit builds during our CircleCI nightly. Fix is required for ios cocoapod release

@atalman merged

@syed-ahmed
Copy link
Contributor

syed-ahmed commented Oct 7, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • N/A (sets environment variable CUDA_MODULE_LOADING='LAZY' to save memory)

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 8, 2022

Link to landed master PR (if applicable):

  • NA

Link to release branch PR:

Criteria Category:

  • CI: Release only change

--
@atalman Merged

@atalman
Copy link
Contributor Author

atalman commented Oct 8, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • CI Critical fix

--
@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 8, 2022

Link to landed master PR (if applicable):

  • NA

Link to release branch PR:

Criteria Category:

  • CI : Release only change, fix RC names

@atalman megred

@EikanWang
Copy link
Collaborator

EikanWang commented Oct 8, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Fix performance regression from previous release.

@malfet: merged as it's part of 3 PRs set that should have been landed in group.

@soulitzer
Copy link
Contributor

soulitzer commented Oct 11, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • bug fix

@malfet: merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 11, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation improvements

@atalman merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 11, 2022

Link to landed master PR (if applicable):

  • N/A

Link to release branch PR:

Criteria Category:

  • Release branch specific changes (change functorch documentation version specifier and links to colab notebooks)

@malfet: merged

@jeffdaily
Copy link
Collaborator

jeffdaily commented Oct 11, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • bug fix

@atalman merged

@jerryzh168
Copy link
Contributor

jerryzh168 commented Oct 11, 2022

@CaoE
Copy link
Collaborator

CaoE commented Oct 12, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:


@atalman merged

@malfet
Copy link
Contributor

malfet commented Oct 12, 2022

Linkt to landed master PR:

Link to release branch PR:

Criteria Category:

  • Documentation improvements / Fixes to regressions against the most recent minor release

@malfet: merged

@malfet
Copy link
Contributor

malfet commented Oct 12, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category:

  • Documentation improvements

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical CI Fix

@atalman merged

@XiaobingSuper
Copy link
Collaborator

XiaobingSuper commented Oct 13, 2022

Link to landed master PR (if applicable):

  • n/a

Link to release branch PR:

Criteria Category:

  • Documentation improvements for quantization

@malfet: This behavior was reverted in #86726, so should it really be documented as default backend?

@zou3519
Copy link
Contributor

zou3519 commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Fixes to regressions against the most recent minor release

@malfet: merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation improvements

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 13, 2022

Link to landed master PR (if applicable):

  • NA

Link to release branch PR:

Criteria Category:

  • [CI] Critical fix

@atalman merged

@BowenBao
Copy link
Collaborator

BowenBao commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • bug fix

@malfet: merged

@BowenBao
Copy link
Collaborator

BowenBao commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • fix regression

@malfet: merged

@BowenBao
Copy link
Collaborator

BowenBao commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Test/CI

@malfet: merged, though it does not sound like a bugfix to me, more test/CI fixes

@BowenBao
Copy link
Collaborator

BowenBao commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • bug fix, regression

@malfet: merged

@smk2007
Copy link
Contributor

smk2007 commented Oct 13, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:
Fix required for enabling the DirectML out-of-tree backend plugin in 1.13.


@malfet: merged

@samdow
Copy link
Contributor

samdow commented Oct 19, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Fixes silent correctness issue (users shouldn't be able to do this and we'll break it in future versions)

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 19, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • CI - Adds more tests

@malfet: Merged. I don't think it's a critical CI fix(as it does not fix anything, just adds more testing), but any new tests can go into the branch

@mikaylagawarecki
Copy link
Contributor

mikaylagawarecki commented Oct 20, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation improvement

@atalman merged

@cpuhrsch
Copy link
Contributor

cpuhrsch commented Oct 20, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category

  • Documentation improvement

@atalman merged

@cpuhrsch
Copy link
Contributor

cpuhrsch commented Oct 20, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category

  • Documentation improvement

@atalman Merged

@cpuhrsch
Copy link
Contributor

cpuhrsch commented Oct 20, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category

  • Documentation improvement / code warning (not exactly sure)

@atalman merged

@cpuhrsch
Copy link
Contributor

cpuhrsch commented Oct 20, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category

  • Documentation improvement

@atalman merged

@zrphercule
Copy link
Contributor

zrphercule commented Oct 20, 2022

Link to landed master PR:

Link to release branch PR:

Criteria Category

  • Fixed asserting bugs in Transformer & added unit test
  • Documentation improvement

@malfet merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 20, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

    1. Documentation improvements

@atalman merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 20, 2022

Link to landed master PR (if applicable):

  • n/a

Link to release branch PR:

Criteria Category:

  • Release branch specific changes. The functorch docs build is not showing the correct version on the release branch. This PR fixes it.

@atalman merged

@george-qi
Copy link
Contributor

george-qi commented Oct 20, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation improvement

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 20, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • CI keep signal green

@malfet: merged

@malfet
Copy link
Contributor

malfet commented Oct 20, 2022

Link to the landed master PR:

Link to the release branch PR:

Criteria Category:

    1. Critical fixes for crashes (crashes on x86 Mac when bitwise not is used on an empty tensor)

@malfet: merged

@zou3519
Copy link
Contributor

zou3519 commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation improvements

@malfet: merged

@malfet
Copy link
Contributor

malfet commented Oct 21, 2022

Link to landed master PRs:

Link to release branch PR:

Criteria Category: Special dispensation (Addressing security concern), needs to be confirmed


@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Test/CI fixes

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical fix

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical Fix

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical Fix

@malfet: merged

@atalman
Copy link
Contributor Author

atalman commented Oct 21, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical fix

@malfet: merged

@malfet
Copy link
Contributor

malfet commented Oct 21, 2022

Link to landed master PRs:

Link to release branch PR:

Criteria Category:

  • Critical fixes for silent correctness

@malfet merged

@atalman
Copy link
Contributor Author

atalman commented Oct 23, 2022

Final RC have been generated. We don't Accept cherry-picks anymore. Please don't post any cherry-picks.

@atalman
Copy link
Contributor Author

atalman commented Oct 24, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Documentation

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 24, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical distributed fix

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 24, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • Critical CUDA/nvrtc fix

@atalman merged

@atalman
Copy link
Contributor Author

atalman commented Oct 24, 2022

Link to landed master PR (if applicable):

Link to release branch PR:

Criteria Category:

  • CI/CD documentation push

@atalman merged

@malfet
Copy link
Contributor

malfet commented Nov 2, 2022

1.13.0 has been published, closing this one.

@malfet malfet closed this as completed Nov 2, 2022
@malfet malfet unpinned this issue Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
oncall: releng In support of CI and Release Engineering triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Development

No branches or pull requests