Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented May 9, 2025

This PR ensure that the default index.html deployed into namespaces web buckets mention Apache OpenServerless and not nuvolaris.

@ghost ghost requested a review from d4rkstar May 9, 2025 11:56
Copy link
Contributor

@d4rkstar d4rkstar left a comment

Choose a reason for hiding this comment

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

Hello,
we probably need to change also the ops util clean command: infact that command will upload a copy of the index.html file. (see https://github.com/apache/openserverless-task/blob/main/util/upload/assets/index.html).

I also planned to implement a better welcome page with OpenServerless logo, font and some instructions on how to use the system and centralize the welcome page on operator or tasks.

@ghost
Copy link
Author

ghost commented May 9, 2025

@d4rkstar I created a similar PR for the task. A better template will requires probably some extra tweaks! Eventually this PR can be deleted, but as I saw that currently OpenServerless stills shows nuvolaris in the html, I thought that a quick and dirty fix was needed! :-D

@d4rkstar
Copy link
Contributor

d4rkstar commented May 9, 2025

@francescotimperi: i remember there are also some tests that will check for the keyword "nuvolaris" in openserverless-testing. May be i'm wrong, but eventually we should check also there :(

@ghost
Copy link
Author

ghost commented May 9, 2025

@d4rkstar indeed. I created also a PR for that!

@d4rkstar d4rkstar merged commit 856f9b1 into apache:main May 9, 2025
1 check passed
sciabarracom pushed a commit to sciabarracom/openserverless-operator that referenced this pull request Jun 10, 2025
chore: wrong containerPort mongo
mobs75 pushed a commit to mobs75/openserverless-operator that referenced this pull request Jul 15, 2025
chore: ensure default index.html deployed into namespace web bucket uses Apache OPS
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