Skip to content

[DO NOT MERGE] Update port from 8081 to 8080 - #33

Open
maysunfaisal wants to merge 7 commits into
devfile-samples:mainfrom
maysunfaisal:update-port-1
Open

[DO NOT MERGE] Update port from 8081 to 8080#33
maysunfaisal wants to merge 7 commits into
devfile-samples:mainfrom
maysunfaisal:update-port-1

Conversation

@maysunfaisal

Copy link
Copy Markdown
Contributor
  • Updates the devfile sample port from 8081 to 8080
  • Doesnt update the attribute alpha.dockerimage-port: 8081 because of backward compatibility in ODC earlier versions. We dont want to backport devfile/library update that far back.

maysunfaisaland others added 3 commits July 10, 2023 14:37
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetk
thepetk changed the base branch from main to v1.2.2October 9, 2023 15:59
@thepetk
thepetk marked this pull request as ready for review October 9, 2023 16:00
@thepetk

thepetk commented Oct 9, 2023

Copy link
Copy Markdown
Contributor

@maysunfaisal I've updated the base of the PR to point to the branch v2.0.0 in order the update of the port to be include to the new version of sample.

I've also added WIP as we need first to merge devfile/registry#219

@thepetkthepetk changed the title Update port from 8081 to 8080WIP: Update port from 8081 to 8080Oct 9, 2023
@thepetk

Copy link
Copy Markdown
Contributor

fixes devfile/api#1112

Signed-off-by: thepetk <thepetk@gmail.com>
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetkthepetk changed the title WIP: Update port from 8081 to 8080Update port from 8081 to 8080Oct 18, 2023
@thepetk
thepetk changed the base branch from v1.2.2 to v2.0.0October 18, 2023 13:59
@thepetk
thepetk changed the base branch from v2.0.0 to mainOctober 23, 2023 12:21
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetkthepetk changed the title Update port from 8081 to 8080[DO NOT MERGE] Update port from 8081 to 8080Oct 23, 2023
@thepetk

Copy link
Copy Markdown
Contributor

@maysunfaisal any chance we could revisit this one and either merge or close?

Sign up for freeto 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

@maysunfaisal@thepetk