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

trying page_size patch #239

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

dericcrago
Copy link
Contributor

@dericcrago dericcrago commented Apr 8, 2021

SUMMARY

testing page_size patch as an alternative to git cloning

COMPONENT NAME
  • tests/utils/shippable/shippable.sh
ADDITIONAL INFO

diff from ansible/ansible#74191 (comment)

@dericcrago dericcrago force-pushed the testing_page_size branch 3 times, most recently from c7ab943 to ab35c55 Compare April 8, 2021 21:00
@felixfontein
Copy link
Collaborator

This is quite a hack, I would avoid merging that as it can cause random problems when the galaxy-related code in Ansible is modified. Since it's limited to CI I guess you have to decide for yourself whether it's worth the trouble :)

@felixfontein
Copy link
Collaborator

The way the patch is organized now is much better, this way it will either be applied completely, or not at all. That should also decrease the probability that it randomly breaks at some point :)

@dericcrago
Copy link
Contributor Author

dericcrago commented Apr 9, 2021

thanks for the comments @felixfontein !

@dericcrago dericcrago merged commit 33c2eb7 into ansible-collections:main Apr 9, 2021
patchback bot pushed a commit that referenced this pull request Apr 9, 2021
(cherry picked from commit 33c2eb7)
dericcrago added a commit that referenced this pull request Apr 9, 2021
(cherry picked from commit 33c2eb7)

Co-authored-by: Deric Crago <[email protected]>
dericcrago added a commit that referenced this pull request Apr 9, 2021
(cherry picked from commit 33c2eb7)
dericcrago added a commit that referenced this pull request Apr 9, 2021
(cherry picked from commit 33c2eb7)

Co-authored-by: Deric Crago <[email protected]>
@dericcrago dericcrago deleted the testing_page_size branch April 15, 2021 20:32
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.

3 participants