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

[BUG] epicli upgrade of cluster created by Epiphany v0.6 fails on [preflight_facts : Store preflight facts] task #1887

Closed
to-bar opened this issue Dec 3, 2020 · 1 comment

Comments

@to-bar
Copy link
Contributor

to-bar commented Dec 3, 2020

Describe the bug
Error message:
PermissionError: [Errno 1] Operation not permitted: /shared/build/06hatodevawrhcalico/vault//../preflight_facts.yml

Cause:

root@vm-bld-node-001:~# ls -la /tmp/build/epi-cli/build/06hatodevawrhcanal/
total 104
drwxr-xr-x  6 operations operations  4096 Dec  3 19:01 .
drwxr-xr-x 84 operations operations 12288 Dec  3 11:22 ..
drwxr-xr-x  5 operations operations  4096 Dec  3 19:01 ansible
drwxr-xr-x  5 operations operations  4096 Dec  3 19:01 backup_1606953697690
-rw-r--r--  1 operations operations  4481 Dec  3 19:01 inventory
-rw-r--r--  1 operations operations 60601 Dec  2 17:54 manifest.yml
-rw-r--r--  1 root       root         446 Dec  2 17:56 preflight_facts.yml
drwxr-xr-x  2 operations operations  4096 Dec  2 17:54 terraform
drwxr-xr-x  2 operations operations  4096 Dec  2 18:32 vault

PR #1825 added the mode parameter to the Store preflight facts task:

- name: Store preflight facts
  delegate_to: localhost
  become: false
  run_once: true
  copy:
    dest: "{{ vault_location }}/../preflight_facts.yml"
    mode: u=rw,g=r,o=

and the become parameter has been added in v0.7.

How to reproduce
Steps to reproduce the behavior:

  1. Deploy cluster using epici v0.6
  2. Run epicli upgrade using code from develop branch.

Expected behavior
epicli upgrade should succeed.

Environment

  • Cloud provider: All
  • OS: All

Additional context
n/a

@przemyslavic
Copy link
Collaborator

The fix for handling a permission issue on preflight_facts.yml works correctly.
The cluster has been successfully upgraded to develop.
Tested:
✔️ 0.6 -> develop
✔️ 0.6 HA -> develop

@mkyc mkyc closed this as completed Dec 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants