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

Use branching date as target_date in branching procedures #340

Closed

Conversation

wbclark
Copy link
Contributor

@wbclark wbclark commented Jan 26, 2024

All procedures currently accept a single target_date. For branching procedures in particular, if the branching date is used as the target_date, this generates correct dates in the section headings but an incorrect date in the command invocation to generate the release procedure. This commit fixes the inconsistency by assuming the branch date is the target_date in branching, and adding four weeks to that date to get the date for GA.

All procedures currently accept a single target_date. For branching
procedures in particular, if the branching date is used as the
target_date, this generates correct dates in the section headings
but an incorrect date in the command invocation to generate the
release procedure. This commit fixes the inconsistency by assuming
the branch date is the target_date in branching, and adding four
weeks to that date to get the date for GA.
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

This doesn't look right. We publish this procedure on the target date and then release Foreman X.Y.0-rc1 on the same day. We also do the same with Katello.

@wbclark
Copy link
Contributor Author

wbclark commented Jan 29, 2024

@ekohl so, using the current cycle as an example, the target_date used to generate the release procedure should be February 20th, even though GA is planned for March 19th, because the target_date is referring to RC1 and not to GA?

Is the RC1 release procedure the only one that is generated in this way?

@ekohl
Copy link
Member

ekohl commented Jan 29, 2024

That is exactly what is happening, so it's all working as intended. Targeted branching date is equal to targeted RC1 date.

@wbclark
Copy link
Contributor Author

wbclark commented Jan 29, 2024

Cool, thanks for the explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants