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

uni06zeta initial version #159

Merged
merged 1 commit into from
Apr 19, 2024
Merged

uni06zeta initial version #159

merged 1 commit into from
Apr 19, 2024

Conversation

afazekas
Copy link
Contributor

@afazekas afazekas commented Apr 3, 2024

uni06zeta initial version

This is based on other peding changes referenced below.

It can produce a passing minimal CI job ATM.
Line-length warning limit extended some config lines are really long,
and less readable when wrapped.

Depends-On: #188

https://github.com/openstack-k8s-operators/architecture/pull/73/files#diff-2b848ad66a53ba3c4f0a6111d518dcf85c4995f7299ee657c76c1a95a0058ebe
184193a
#83

@afazekas afazekas force-pushed the zeta branch 6 times, most recently from 1a5e5f2 to a342334 Compare April 10, 2024 11:15
@afazekas afazekas changed the title [DRAFT] uni06zeta draft uni06zeta initial version Apr 12, 2024
@afazekas afazekas marked this pull request as ready for review April 12, 2024 07:40
@afazekas afazekas force-pushed the zeta branch 3 times, most recently from c0d93f1 to 46b8b56 Compare April 15, 2024 07:50
automation/vars/uni06zeta.yaml Outdated Show resolved Hide resolved
automation/vars/uni06zeta.yaml Outdated Show resolved Hide resolved
sdatko added a commit to sdatko/openstack-k8s-operators-architecture that referenced this pull request Apr 15, 2024
sdatko added a commit to sdatko/openstack-k8s-operators-architecture that referenced this pull request Apr 15, 2024
sdatko added a commit to sdatko/openstack-k8s-operators-architecture that referenced this pull request Apr 17, 2024
sdatko added a commit to sdatko/openstack-k8s-operators-architecture that referenced this pull request Apr 17, 2024
@afazekas afazekas force-pushed the zeta branch 7 times, most recently from c4abfe7 to 36db2eb Compare April 18, 2024 12:35
@afazekas
Copy link
Contributor Author

I removing the hook candidates they never meant to be the final version,
some test still can pass, they might came back in another revision or not at all.

Copy link
Contributor

@fultonj fultonj left a comment

Choose a reason for hiding this comment

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

Overall this is pretty good. I see a few changes but then I think we'll be OK to merge.

examples/dt/uni06zeta/control-plane.md Outdated Show resolved Hide resolved
examples/dt/uni06zeta/control-plane/nncp/values.yaml Outdated Show resolved Hide resolved
examples/dt/uni06zeta/README.md Outdated Show resolved Hide resolved
cd architecture/examples/dt/uni06zeta
```

Edit [controle-plane/service-values.yaml](control-plane/service-values.yaml) and
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Edit [controle-plane/service-values.yaml](control-plane/service-values.yaml) and
Edit [control-plane/service-values.yaml](control-plane/service-values.yaml) and

```

Edit [controle-plane/service-values.yaml](control-plane/service-values.yaml) and
[nncp/values.yaml](nncp/values.yaml).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[nncp/values.yaml](nncp/values.yaml).
[control-plane/nncp/values.yaml](control-plane/nncp/values.yaml).

This is based on other peding changes referenced below.
Line-length warning limit extended some config lines are really long,
and less readable when wrapped.

https://github.com/openstack-k8s-operators/architecture/pull/73/files#diff-2b848ad66a53ba3c4f0a6111d518dcf85c4995f7299ee657c76c1a95a0058ebe
184193a
#83
Copy link
Contributor

@abays abays left a comment

Choose a reason for hiding this comment

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

/lgtm

Let's see what @fultonj thinks and then we can merge

@fultonj fultonj merged commit 6747987 into main Apr 19, 2024
4 checks passed
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.

5 participants