Skip to content

Simplify GitLab integration per RHTAP-6483#138

Merged
Roming22 merged 1 commit intoredhat-appstudio:mainfrom
gtrivedi88:docs/address-sme-review
Apr 13, 2026
Merged

Simplify GitLab integration per RHTAP-6483#138
Roming22 merged 1 commit intoredhat-appstudio:mainfrom
gtrivedi88:docs/address-sme-review

Conversation

@gtrivedi88
Copy link
Copy Markdown
Contributor

@gtrivedi88 gtrivedi88 commented Apr 13, 2026

Summary

  • RHTAP-6483: Replace manual oc create secret / oc label / oc annotate commands with tsf integration gitlab CLI command. The manual commands fail because the tenant namespace does not exist at that point in the install flow. The installer handles namespace and secret creation for default-tenant during deployment.
  • PR Address SME feedback across all documentation pages #134 review feedback: Remove GH App vs GL webhook detail from CI checks. Add "For GitLab, the webhook is created and configured in the repository."

Test plan

  • Verify tsf integration gitlab command matches installer behavior
  • Verify oc get secret tsf-gitlab-integration -n tsf returns the secret after deployment

RHTAP-6483: Replace manual oc create/label/annotate secret commands
with tsf integration gitlab CLI command. The installer handles
namespace and secret creation for default-tenant during deployment.

PR redhat-appstudio#134 review: Remove GH App vs GL webhook detail from CI checks
description. Add GitLab webhook note per SME feedback.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gtrivedi88 gtrivedi88 requested review from Roming22 and jkopriva April 13, 2026 09:22
appstudio.redhat.com/scm.host="$GITLAB__HOST"
tsf integration gitlab \
--group="$GITLAB__GROUP" \
--host="$GITLAB__HOST" \
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would put here some example, that is must be without https:.... for example: --host="gitlab.com"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is already specified in https://redhat-appstudio.github.io/tsf-cli/tsf/preparing-to-install.html#steps, where all vars are documented.

Copy link
Copy Markdown
Member

@Roming22 Roming22 left a comment

Choose a reason for hiding this comment

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

/lgtm

@Roming22 Roming22 merged commit bc378cd into redhat-appstudio:main Apr 13, 2026
2 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.

3 participants