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

Please make the build reproducible #51419

Closed
wants to merge 1 commit into from

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Jan 29, 2019

Whilst working on the Reproducible Builds effort, we noticed that ansible could not be built reproducibly, mostly due to variations in the documentation between successive builds.

This pull request includes a number of changes to make it build in a determinstic manner. Currently it still varies a bit even with this patch, presumably due to nondetermistic filesystem ordering but it will be easier to see after these are merged.

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Jan 29, 2019
@lamby
Copy link
Contributor Author

lamby commented Jan 29, 2019

(Force-pushed to drop one of the patches after rebasing from a devel from this century.)

@ansibot ansibot added small_patch and removed needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html labels Jan 29, 2019
@ansibot

This comment has been minimized.

@ansibot

This comment has been minimized.

@dagwieers dagwieers added the docs This issue/PR relates to or includes documentation. label Jan 29, 2019
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Jan 29, 2019
@jctanner
Copy link
Contributor

@lamby What artifacts are you producing to assert the build is reproducible? Python dist tarball, rpm, deb, ?

I'm not sure we can really accept this without some sort of ability to verify the behavior on our side as well.

needs_info

@ansibot ansibot added the needs_info This issue requires further information. Please answer any outstanding questions. label Jan 29, 2019
@lamby
Copy link
Contributor Author

lamby commented Jan 29, 2019

@ansibot ansibot removed the needs_info This issue requires further information. Please answer any outstanding questions. label Jan 29, 2019
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Feb 6, 2019
bcoca added a commit to bcoca/ansible that referenced this pull request Feb 18, 2019
bcoca added a commit that referenced this pull request Feb 18, 2019
@ansibot ansibot removed the core_review In order to be merged, this PR must follow the core review workflow. label Feb 22, 2019
@ansibot ansibot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Feb 22, 2019
@acozine
Copy link
Contributor

acozine commented Mar 21, 2019

I'm removing the docs label as this doesn't appear to be a documentation PR. Feel free to ping me for input if it would be helpful.

@acozine acozine removed the docs This issue/PR relates to or includes documentation. label Mar 21, 2019
@sivel
Copy link
Member

sivel commented Mar 21, 2019

This was already resolved by #51453

If you are still having problems, please submit a new issue.

If you have further questions please stop by IRC or the mailing list:

@sivel sivel closed this Mar 21, 2019
@ansible ansible locked and limited conversation to collaborators Jul 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. small_patch stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants