-
Notifications
You must be signed in to change notification settings - Fork 9
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
Jenkins file to build oracle docker image #5
base: bellecour
Are you sure you want to change the base?
Conversation
oracle/Dockerfile
Outdated
COPY yarn.lock . | ||
RUN cd oracle && yarn remove web3 web3-utils && cd .. | ||
RUN NOYARNPOSTINSTALL=1 yarn install --frozen-lockfile --production | ||
RUN cd commons && yarn add [email protected] && cd .. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Is it possible to
yarn upgrade [email protected]
instead ofremove
&add
? - I don't remember, do we need
&& cd ..
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to try the point 1 but for 2. since we are working on 2 different package.json
file (oracle / commons) I think we need it. But gonna try it as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A good approach here might be something like
WORKDIR /bla
RUN bla bla
WORKDIR /blu
RUN blu blu
(?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I just tried
upgrade
it does not work, I have the same issue msg of the beginning - Going to implement the WORKDIR thing
oracle/docker-compose.yml
Outdated
build: | ||
context: .. | ||
dockerfile: oracle/Dockerfile | ||
image : docker-regis.iex.ec/oracle:feature-ef980a52 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe
docker-regis.iex.ec/oracle:$ORACLE_TAG
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
Jenkinsfile
Outdated
imageprivacy: 'private', | ||
dockerImageRepositoryName : 'oracle' | ||
) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing end-of-line :)
Same in (oracle/.env.example)
Co-authored-by: Jérémy James Toussaint <[email protected]>
…ExecBlockchainComputing/tokenbridge into feature/bellecour-build-oracle
oracle/Dockerfile
Outdated
|
||
WORKDIR /mono/oracle | ||
RUN [email protected] [email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RUN [email protected] [email protected] | |
RUN yarn upgrade [email protected] [email protected] |
I think*
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes mb (just saw it)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved for this first milestone :) To be merged later I guess (?)
You can find the job jenkins => Token-bridge