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

[cloud_infra_center] none-root user for bastion, haproxy,DNS reboot i… #171

Merged
merged 2 commits into from
Oct 18, 2022

Conversation

juniororange
Copy link

none-root user for bastion,haproxy ,dns reboot issue.

@@ -68,6 +68,17 @@ _etcd-server-ssl._tcp IN SRV 0 10 2380 etcd-0.openshift.second.com.
IN SRV 0 10 2380 etcd-2.openshift.second.com.
```

equired DNS records
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required not equired

get_url:
url: "{{ openshift_client_url }}/openshift-client-linux.tar.gz"
dest: "{{ openshift_install_dir }}/openshift-client-linux.tar.gz"
mode: 0755
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the mode of installer file when removed 0755?

Copy link
Author

@juniororange juniororange Oct 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need 0755 in system ,umask as default is 0022, so in the root user the image download will be as 0755 and in the non-root user the image will belong to user. If put 0755 in it , it need to use sudo to handle the file.

chengz added 2 commits October 18, 2022 16:57
…ssue

Signed-off-by: chengz <chengz@cn.ibm.com>
Signed-off-by: chengz <chengz@cn.ibm.com>
@zhengxiaomei123 zhengxiaomei123 merged commit ddeb2b6 into IBM:main Oct 18, 2022
frankmit11 pushed a commit to frankmit11/icic-ocp-upi that referenced this pull request Feb 9, 2023
[cloud_infra_center] none-root user for bastion, haproxy,DNS reboot i…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants