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

Update quickstarts to use Glow #895

Merged
merged 36 commits into from
May 13, 2024
Merged

Conversation

kabir
Copy link
Contributor

@kabir kabir commented Apr 13, 2024

No description provided.

kabir and others added 27 commits January 17, 2024 15:54
Port enhanced quickstarts to Glow
Add module
Develop demo app
Document QS
Add micrometer and opentelemetry with Glow support
Sync Glow Preview branch with 31.0.0.final
Add glow rendering to README-source of enhanced quickstarts
Also upgrade to 5.0.0.Final of the WildFly maven plugin
@kabir kabir requested a review from emmartins as a code owner April 13, 2024 15:49
@kabir
Copy link
Contributor Author

kabir commented Apr 14, 2024

/retest

wildfly-branch: ${{ github.base_ref }}
# There is no upstream glow-preview branch. The QS glow-preview branch targets WildFLy main
#wildfly-branch: ${{ github.base_ref }}
wildfly-branch: main
Copy link
Contributor

Choose a reason for hiding this comment

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

I think now should be reverted?

wildfly-repo: "wildfly/wildfly"

Test-build-with-deps-matrix:
name: BUILD WITH DEPS - JDK${{ matrix.jdk }} - ${{ matrix.os }}
runs-on: ${{ matrix.os }}
needs: WildFly-build
if:
Copy link
Contributor

Choose a reason for hiding this comment

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

why this change?

wildfly-branch: ${{ github.base_ref }}
# There is no upstream glow-preview branch. The QS glow-preview branch targets WildFLy main
#wildfly-branch: ${{ github.base_ref }}
wildfly-branch: main
Copy link
Contributor

Choose a reason for hiding this comment

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

I think now should be reverted?

@@ -20,7 +20,8 @@ env:
MATRIX_JDK: '"11", "17"'
MATRIX_OS: '"ubuntu-latest"'
SERVER_PROVISIONING_SERVER_HOST: 'http://localhost'

EXTRA_RUN_ARGS: '-Pprovisioned-server -DpostgresqlUsername="test" -DpostgresqlPassword="test"'
Copy link
Contributor

Choose a reason for hiding this comment

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

-Pprovisioned-server as a run arg looks weird, may come back after checking the QS changes

@@ -132,7 +133,9 @@ jobs:
WildFly-build:
uses: wildfly/wildfly/.github/workflows/shared-wildfly-build.yml@main
with:
wildfly-branch: ${{ github.base_ref }}
# There is no upstream glow-preview branch. The QS glow-preview branch targets WildFLy main
Copy link
Contributor

Choose a reason for hiding this comment

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

I think now should be reverted?

@@ -3,7 +3,7 @@ name: WildFly Quickstarts Update 'README.adoc's
on:
push:
branches:
- main
- glow-preview
Copy link
Contributor

Choose a reason for hiding this comment

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

I think now should be reverted?

README.adoc Outdated
@@ -62,7 +62,7 @@ ifndef::ProductRelease,EAPXPRelease[]
:productImageVersion: 31.0
:githubRepoUrl: https://github.com/wildfly/quickstart/
:githubRepoCodeUrl: https://github.com/wildfly/quickstart.git
:WildFlyQuickStartRepoTag: 31.0.0.Final
:WildFlyQuickStartRepoTag: 31.0.0.Final-glow-preview
Copy link
Contributor

Choose a reason for hiding this comment

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

Please note that any change in README.adoc that was not auto done by the asciidoc-reducer should be moved to related README-source.adoc . I think you should confirm this, and revert all README.adoc

I guess those portedToGlow should now be removed and the shared-docs include file should be updated, wrt the XML snippet change for both server provisioning and bootable jar

Copy link

openshift-ci bot commented May 7, 2024

@kabir: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/wildfly-quickstart-e2e 8fd8158 link true /test wildfly-quickstart-e2e

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@emmartins emmartins merged commit a716e95 into wildfly:main May 13, 2024
511 of 516 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.

4 participants