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

setup_botocore_pip: create tempdir in /var/tmp #734

Conversation

goneri
Copy link
Member

@goneri goneri commented Mar 24, 2022

Depends-On: ansible/ansible-zuul-jobs#1418
Depends-On: #739
Depends-On: #739
Depends-On: #740

/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.

See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs

/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.

See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs
@tremble
Copy link
Contributor

tremble commented Mar 24, 2022

Will need updating in community.aws too. If you happen to know how we could share this between the collections that'd be good.

@alinabuzachis
Copy link
Contributor

@goneri Is the splitter's behaviour expected?

@goneri
Copy link
Member Author

goneri commented Mar 24, 2022

@goneri Is the splitter's behaviour expected?

I think it does. There is no logic in place to know if a test change impact another test.

@goneri
Copy link
Member Author

goneri commented Mar 24, 2022

Will need updating in community.aws too. If you happen to know how we could share this between the collections that'd be good.

As far as I know, there is nothing like this in place.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@goneri
Copy link
Member Author

goneri commented Mar 24, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@alinabuzachis
Copy link
Contributor

recheck

@alinabuzachis alinabuzachis reopened this Mar 25, 2022
@alinabuzachis alinabuzachis added backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch labels Mar 25, 2022
@softwarefactory-project-zuul
Copy link
Contributor

@alinabuzachis
Copy link
Contributor

recheck

@softwarefactory-project-zuul
Copy link
Contributor

@alinabuzachis
Copy link
Contributor

@goneri /bin/sh: line 1: /usr/bin/ansible-galaxy: No such file or directory

@goneri
Copy link
Member Author

goneri commented Mar 25, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@goneri
Copy link
Member Author

goneri commented Mar 25, 2022

recheck

@goneri goneri mentioned this pull request Mar 25, 2022
1 task
@softwarefactory-project-zuul
Copy link
Contributor

This change depends on a change that failed to merge.

@goneri
Copy link
Member Author

goneri commented Mar 25, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@goneri
Copy link
Member Author

goneri commented Mar 25, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Mar 28, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit aeba19a into ansible-collections:main Mar 28, 2022
@patchback
Copy link

patchback bot commented Mar 28, 2022

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/aeba19a68646018215ee76ebf39c654c21dac595/pr-734

Backported as #745

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 28, 2022
setup_botocore_pip: create tempdir in /var/tmp

Depends-On: ansible/ansible-zuul-jobs#1418
Depends-On: #739
Depends-On: #739
Depends-On: #740
/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.
See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit aeba19a)
@patchback
Copy link

patchback bot commented Mar 28, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/aeba19a68646018215ee76ebf39c654c21dac595/pr-734

Backported as #746

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 28, 2022
setup_botocore_pip: create tempdir in /var/tmp

Depends-On: ansible/ansible-zuul-jobs#1418
Depends-On: #739
Depends-On: #739
Depends-On: #740
/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.
See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit aeba19a)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 28, 2022
[PR #734/aeba19a6 backport][stable-3] setup_botocore_pip: create tempdir in /var/tmp

This is a backport of PR #734 as merged into main (aeba19a).
Depends-On: ansible/ansible-zuul-jobs#1418
Depends-On: #739
Depends-On: #739
Depends-On: #740
/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.
See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 29, 2022
[PR #734/aeba19a6 backport][stable-2] setup_botocore_pip: create tempdir in /var/tmp

This is a backport of PR #734 as merged into main (aeba19a).
Depends-On: ansible/ansible-zuul-jobs#1418
Depends-On: #739
Depends-On: #739
Depends-On: #740
/tmp is based on ramfs and is not suitable for large files.
/var/tmp should be used instead.
See: https://fedoraproject.org/wiki/Features/tmp-on-tmpfs
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…_health_check

route53_health_check - boto3 migration 

SUMMARY
route53_health_check migration to boto3
Depends-On: ansible-collections#746
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
route53_health_check
ADDITIONAL INFORMATION
Todo:

 changelog
 docs for new return values
 tests for return values

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…_health_check

route53_health_check - boto3 migration 

SUMMARY
route53_health_check migration to boto3
Depends-On: ansible-collections#746
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
route53_health_check
ADDITIONAL INFORMATION
Todo:

 changelog
 docs for new return values
 tests for return values

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch community_review has_issue integration tests/integration mergeit Merge the PR (SoftwareFactory) needs_triage small_patch Hopefully easy to review tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants