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 pipeline.yml for new Artifactory url #129

Merged
merged 9 commits into from
Sep 20, 2019
Merged

Update pipeline.yml for new Artifactory url #129

merged 9 commits into from
Sep 20, 2019

Conversation

jesusHCG
Copy link
Contributor

Description

Change of Artifactory url and docker image

Related GitHub issue or JIRA story

https://jira.autodesk.com/browse/DYN-2071
https://jira.autodesk.com/browse/DYN-2072

Checklist

  • PR title reflects the changes it introduces. Consider using this template: QNTM-XXXX: Implement X Feature
  • [ - ] Code compiles correctly
  • [ - ] No linting errors
  • [ - ] All tests passing
  • Updated README.md / documentation (if necessary)

pipeline.yml Outdated Show resolved Hide resolved
@jesusHCG
Copy link
Contributor Author

Hi @QilongTang, @mjkkirschner and @pinzart90, We already have a new docker image, can you help us validate this pull request to do some test?

@QilongTang
Copy link
Contributor

@jesusHCG I am going to merge and request you test on the VM if that is OK, let me know!

@QilongTang QilongTang merged commit 1077814 into DynamoDS:develop Sep 20, 2019
nodejs_version: "10"

build:
customized_builder_image:
registry: artifactory
url: autodesk-docker.art-bobcat.autodesk.com/quantum/hardened-nodejs:latest
url: artifactory.dev.adskengineer.net/docker-local-v2/docker-local-v2/dynamo/dynamolibrariejs:latest
Copy link
Collaborator

@pinzart90 pinzart90 Sep 20, 2019

Choose a reason for hiding this comment

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

What's up with the path to the image ?
/docker-local-v2/docker-local-v2
@jesusHCG @mjkkirschner ?
seems like this works too artifactory.dev.adskengineer.net/docker-local-v2/dynamo/dynamolibrariejs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i used the url from the artifactory

image

QilongTang added a commit that referenced this pull request Mar 12, 2020
* Fix tests (#124)

* Fix test

* Update README.md

* Forcing version by package.json

* Add pipeline to run test in docker image. (#125)

* Add pipeline to run test in docker image.

* Create .npmrc

* Update README.md

* Update README.md

* Update README.md

* Update package.json

* Update package.json

* Update Jenkinsfile

* Slack notification and docker suport (#126)

* Delete duplicate folder (#127)

* Add missing files

* Delete indent-arrow-down-wo-lines.svg

* Delete indent-arrow-right-wo-lines.svg

* Delete plus-symbol.svg

* Update package.json (#128)

* Update pipeline.yml for new Artifactory url (#129)

* Update package.json

* Update pipeline.yml

* Update pipeline.yml

* Update pipeline.yml

* Update pipeline.yml

* Update pipeline.yml

* Update pipeline.yml

* New docker image

* Add of csproj file for nuget packing (#130)

* add of csproj file for nuget packing

* New sources for dotnet pack

* Update pipeline.yml

* Update pipeline.yml

* Update pipeline.yml

* Update librariejs.csproj

* Update librariejs.csproj

* Update librariejs.csproj

* Change name of project

* new NUGET_API_ID (#131)

* CI to switch to use stable branch (#136)

* Create .sonarcloud.properties

* Trident web browser control compat (#133)

* cache the old url when the image fails to load - we need to be able to look it up later to reload the correct image.
add core-js for ie polyfills (startsWith) - we can try just using a polyfill for that method only later.
add core-js to entrypoint so its imported

* remove discarded changes for extra event

* Changes to use cbp/stable branch for the librariejs

* Changes to use the CI Lib  using the credential feature

Co-authored-by: Aaron (Qilong) <[email protected]>
Co-authored-by: Michael Kirschner <[email protected]>

* Change of credential ID to use custom docker image (#137)

* Develop (#139)

* Change of credential ID to use custom docker image

* Remove additional line of nprmc

Co-authored-by: jesusHCG <[email protected]>
Co-authored-by: manuelsaldivar525 <[email protected]>
Co-authored-by: Aaron (Qilong) <[email protected]>
Co-authored-by: Michael Kirschner <[email protected]>
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