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

ignore failure with old Python #590

Conversation

goneri
Copy link
Member

@goneri goneri commented Jun 2, 2021

We only support Python 3.8 now. This commit had:

  • a script to refresh the ignore files:
    tests/sanity/refresh_ignore_files
  • the ignore files.

See: ansible-collections/amazon.aws#381

We only support Python 3.8 now. This commit had:

- a script to refresh the ignore files:
  tests/sanity/refresh_ignore_files
- the ignore files.

See: ansible-collections/amazon.aws#381
@ansibullbot
Copy link

@goneri: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot ansibullbot added needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage tests tests labels Jun 2, 2021
@jillr jillr added gate and removed needs_triage labels Jun 2, 2021
@ansible-zuul ansible-zuul bot merged commit ba89813 into ansible-collections:main Jun 2, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
alinabuzachis added a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
Fix tempfolder path since Fedora uses tmpfs

SUMMARY

This is a fix for
2021-12-20 10:36:56.079435 \| fedora-35 \| TASK [aws_s3 : make tempfile 4 GB for linux] ***********************************
--
5391 | 2021-12-20 10:36:56.079637 \| fedora-35 \| task path: /home/zuul/.ansible/collections/ansible_collections/amazon/aws/tests/integration/targets/aws_s3/tasks/main.yml:499
5392 | 2021-12-20 10:36:56.244626 \| fedora-35 \| Using module file /tmp/ansible-test-qyqzaib3/ansible/modules/command.py
5393 | 2021-12-20 10:36:56.244945 \| fedora-35 \| Pipelining is enabled.
5394 | 2021-12-20 10:36:56.245217 \| fedora-35 \| <testhost> ESTABLISH LOCAL CONNECTION FOR USER: zuul
5395 | 2021-12-20 10:36:56.245359 \| fedora-35 \| <testhost> EXEC /bin/sh -c 'ANSIBLE_DEBUG_BOTOCORE_LOGS=True /home/zuul/venv/bin/python && sleep 0'
5396 | 2021-12-20 10:36:57.054748 \| fedora-35 \| fatal: [testhost]: FAILED! => {
5397 | 2021-12-20 10:36:57.054840 \| fedora-35 \|     "changed": true,
5398 | 2021-12-20 10:36:57.054850 \| fedora-35 \|     "cmd": [
5399 | 2021-12-20 10:36:57.054856 \| fedora-35 \|         "dd",
5400 | 2021-12-20 10:36:57.054862 \| fedora-35 \|         "if=/dev/zero",
5401 | 2021-12-20 10:36:57.054868 \| fedora-35 \|         "of=/tmp/ansible.6d71q1je/largefile",
5402 | 2021-12-20 10:36:57.054875 \| fedora-35 \|         "bs=1M",
5403 | 2021-12-20 10:36:57.054880 \| fedora-35 \|         "count=4096"
5404 | 2021-12-20 10:36:57.054886 \| fedora-35 \|     ],
5405 | 2021-12-20 10:36:57.054892 \| fedora-35 \|     "delta": "0:00:00.601860",
5406 | 2021-12-20 10:36:57.054897 \| fedora-35 \|     "end": "2021-12-20 10:36:57.021668",
5407 | 2021-12-20 10:36:57.054903 \| fedora-35 \|     "invocation": {
5408 | 2021-12-20 10:36:57.054909 \| fedora-35 \|         "module_args": {
5409 | 2021-12-20 10:36:57.054915 \| fedora-35 \|             "_raw_params": "dd if=/dev/zero of=/tmp/ansible.6d71q1je/largefile bs=1M count=4096",
5410 | 2021-12-20 10:36:57.054928 \| fedora-35 \|             "_uses_shell": false,
5411 | 2021-12-20 10:36:57.054935 \| fedora-35 \|             "argv": null,
5412 | 2021-12-20 10:36:57.054941 \| fedora-35 \|             "chdir": null,
5413 | 2021-12-20 10:36:57.054947 \| fedora-35 \|             "creates": null,
5414 | 2021-12-20 10:36:57.054953 \| fedora-35 \|             "executable": null,
5415 | 2021-12-20 10:36:57.054960 \| fedora-35 \|             "removes": null,
5416 | 2021-12-20 10:36:57.054966 \| fedora-35 \|             "stdin": null,
5417 | 2021-12-20 10:36:57.054972 \| fedora-35 \|             "stdin_add_newline": true,
5418 | 2021-12-20 10:36:57.054978 \| fedora-35 \|             "strip_empty_ends": true,
5419 | 2021-12-20 10:36:57.054984 \| fedora-35 \|             "warn": false
5420 | 2021-12-20 10:36:57.055001 \| fedora-35 \|         }
5421 | 2021-12-20 10:36:57.055007 \| fedora-35 \|     },
5422 | 2021-12-20 10:36:57.055012 \| fedora-35 \|     "msg": "non-zero return code",
5423 | 2021-12-20 10:36:57.055017 \| fedora-35 \|     "rc": 1,
5424 | 2021-12-20 10:36:57.055023 \| fedora-35 \|     "start": "2021-12-20 10:36:56.419808",
5425 | 2021-12-20 10:36:57.055030 \| fedora-35 \|     "stderr": "dd: error writing '/tmp/ansible.6d71q1je/largefile': No space left on device\n947+0 records in\n946+0 records out\n992100352 bytes (992 MB, 946 MiB) copied, 0.596905 s, 1.7 GB/s",
5426 | 2021-12-20 10:36:57.055037 \| fedora-35 \|     "stderr_lines": [
5427 | 2021-12-20 10:36:57.055043 \| fedora-35 \|         "dd: error writing '/tmp/ansible.6d71q1je/largefile': No space left on device",
5428 | 2021-12-20 10:36:57.055049 \| fedora-35 \|         "947+0 records in",
5429 | 2021-12-20 10:36:57.055054 \| fedora-35 \|         "946+0 records out",
5430 | 2021-12-20 10:36:57.055060 \| fedora-35 \|         "992100352 bytes (992 MB, 946 MiB) copied, 0.596905 s, 1.7 GB/s"
5431 | 2021-12-20 10:36:57.055066 \| fedora-35 \|     ],
5432 | 2021-12-20 10:36:57.055071 \| fedora-35 \|     "stdout": "",
5433 | 2021-12-20 10:36:57.055077 \| fedora-35 \|     "stdout_lines": []
5434 | 2021-12-20 10:36:57.055083 \| fedora-35 \| }


ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

aws_s3

Reviewed-by: None <None>
Reviewed-by: Gonéri Le Bouder <[email protected]>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants