Giter Club home page Giter Club logo

Comments (12)

ThomWright avatar ThomWright commented on August 15, 2024

This looks like it might have been fixed in commit 1eef9e9. Could you confirm?

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Tom, I'm pretty sure that's a different problem. In this case the app was deployed successfully and that error appeared in the log. While without the 1eef9e9 fix the app would not even deploy.

from ibm-websphere-liberty-buildpack.

ThomWright avatar ThomWright commented on August 15, 2024

Jarek, I'm not sure I'm following. This issue was raised a few weeks before that commit. How could the app have successfully deployed?

I've just tried pushing the application and have been unable to reproduce the problem.

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Sorry, I was trying to say the there are two separate problems related to Spring. 1eef9e9 fixed one problem but the problem in this issue is differnet. The problem in this issue is that the application got deployed OK but there is an error in the log while it's running (looks like a spring autoconfiguration problem). The problem that I fixed was a case where the application wouldn't even deploy. That was a problem a problem in the detect scripts.

from ibm-websphere-liberty-buildpack.

ryanjbaxter avatar ryanjbaxter commented on August 15, 2024

Do you still need me to test or are you satisfied this is a separate issue?

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Can you send the app or point us to it?

from ibm-websphere-liberty-buildpack.

ryanjbaxter avatar ryanjbaxter commented on August 15, 2024

I provided a war to @ThomWright earlier today.

from ibm-websphere-liberty-buildpack.

ThomWright avatar ThomWright commented on August 15, 2024

Ryan and Jarek,

I'm still trying to understand what the problem is here. This is what I understand so far:

The .war has a 'Main-Class' entry in the MANIFEST.MF file. Now, with Jarek's changes, the Liberty buildpack runs the app as if it was a .jar. It then runs on the embedded instance of Tomcat. This works fine.

From what I understand, the app has also been packaged without the 'Main-Class' entry in the manifest, so the buildpack will run the app as a .war on the Liberty Profile server. This is when the problem occurs.

Please correct me if I've misunderstood.

I've tried running the app sans the 'Main-Class' in the manifest on both the Liberty and Java buildpacks. I saw exceptions on both. Neither matched the exception in this issue. Is there something else I should be doing to reproduce this?

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Tom,

Interesting. Could you try deploying this application with buildpack code at revision before the fix for Main-Class went it? Maybe it was taken care of by the Main-Class fix but I'm not convinced yet.

Ryan,

Are you able to still reproduce the problem on your end?

from ibm-websphere-liberty-buildpack.

ryanjbaxter avatar ryanjbaxter commented on August 15, 2024

@jgawor I just got back to trying this and it is working now if I specify the url to the buildpack. If I just push the war without specifying the buildpack Bluemix chooses to use the CF Java buildpack as oppose to using Liberty, but that doesn't seem to be a buildpack problem.

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Getting back to this... I was able to test this issue with the original application. The application was .war file with META-INF/MANIFEST.MF with Main-Class attribute set. I was unable to replicate the original problem (Failed to access resource) but I do see the same behavour as @ryanjbaxter where the Java buildpack picks up this application. That is happening becuase the Liberty buildpack detect method returned "Application can not be run by more than one container" type of error and Cloud Foundry continued to look for other buildpacks that can handle this type of application. In which case, the Java buildpack picked it up next and things deployed fine.
The 1eef9e9 fix addresses the "Application can not be run by more than one container" error and therefore, next time when this type of application is deployed, the Liberty buildpack should pick it up instead of the Java buildpack.

from ibm-websphere-liberty-buildpack.

jgawor avatar jgawor commented on August 15, 2024

Closing this issue as the core problem should be fixed now.

from ibm-websphere-liberty-buildpack.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.