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.

Sign up for GitHub

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

Jump to bottom

Add horizon charts #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add horizon charts #34

wants to merge 1 commit into from

Conversation

fightdou
Copy link
Contributor

No description provided.

@fightdou fightdou force-pushed the horizon branch 4 times, most recently from ef22cd2 to 585bcfb Compare July 29, 2022 05:54
jeffyjf
jeffyjf reviewed Jul 30, 2022
View reviewed changes
Copy link
Contributor

@jeffyjf jeffyjf left a comment

Choose a reason for hiding this comment

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

charts/horizon/templates/conf/_custom_local_settings.tpl 这是各空文件，是故意留的吗？

jeffyjf
jeffyjf requested changes Jul 30, 2022
View reviewed changes
charts/horizon/values.yaml Outdated

## @section Image Parameters
## @param imageRegistry [t#Image Registry] The registry address of openstack kolla image
imageRegistry: "docker.io"
Copy link
Contributor

Choose a reason for hiding this comment

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

imageRegistry 默认改成 registry.aliyuncs.com

Copy link
Contributor Author

Choose a reason for hiding this comment

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

horizon 我用ubuntu-source-horizon这个镜像安装出来的图片显示异常，用ubuntu-binary-horizon这类镜像装出来显示是正常的，阿里云没有同步这类镜像，所以用的官方的

@fightdou fightdou force-pushed the horizon branch 2 times, most recently from 736baec to 0568169 Compare July 31, 2022 13:56
@fightdou
Copy link
Contributor Author

charts/horizon/templates/conf/_custom_local_settings.tpl 这是各空文件，是故意留的吗？

这个文件是必须存在的，不存在镜像启动会报错

@fightdou fightdou force-pushed the horizon branch 4 times, most recently from 4d2a5fc to 5955cc0 Compare August 1, 2022 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeffyjf jeffyjf jeffyjf requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fightdou @jeffyjf