Skip to content

Fix namespace conflicts and snap compatibility in deployment pipeline#5

Open
motangpuar wants to merge 1 commit intoo-ran-sc:masterfrom
motangpuar:master
Open

Fix namespace conflicts and snap compatibility in deployment pipeline#5
motangpuar wants to merge 1 commit intoo-ran-sc:masterfrom
motangpuar:master

Conversation

@motangpuar
Copy link
Copy Markdown

@motangpuar motangpuar commented Jun 5, 2025

  • Change helm install to helm upgrade --install, so these scripts
    won't stop upon finding that the namespace exists
  • Change yq being installed, since snap won't run on containered version
    of ubuntu (Test on K8S jenkins->k8s pipeline)
  • Parameterized env.REPO on jenkins manual-start-pipeline (Just in case I want to
    test my own fork instead of yours)

Tried to request for code review through gerrit, but I got 403.
My gerrit account name is ymarselino (m11202819@gapps.ntust.edu.tw) affiliated with OSC Lab Taiwan (NTUST)

image

@JohnKeeneyEST
Copy link
Copy Markdown
Contributor

JohnKeeneyEST commented Jun 5, 2025

Hi @motangpuar

Unfortunately, this Github version of IT/DEP is a read-only mirror of the repo maintained in O-RAN SC Gerrit https://gerrit.o-ran-sc.org/r/q/project:it/dep
You will need to create a gerrit review there.
To push your review you may first need set up an Linux Foundation ID and agree to the O-RAN SC contributor agreement.
See https://lf-o-ran-sc.atlassian.net/wiki/spaces/ORAN/pages/14386333/Project+Developer+Wiki

Meanwhile, we will have a look at your suggestions.
Thanks.

@motangpuar
Copy link
Copy Markdown
Author

Thank you @JohnKeeneyEST I'll try to push the review on gerrit

@motangpuar motangpuar force-pushed the master branch 6 times, most recently from 9a802e9 to 8a92b23 Compare June 11, 2025 07:36
- Replace `helm install` to `helm upgrade --install` (proceed when ns
  exist)
- Install `yq` by download the binary instead of snap (no snap in
  container)
- Jenkins pipeline env.REPO

Change-Id: I346617ee9f593d16a9afcd5bbce397005899ee30
Signed-off-by: ymarselino <m11202819@gapps.ntust.edu.tw>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants